Cannot build runtime repo on Windows with ninja

[vitek-karas]

Clean clone of the runtime repo, Windows 10, latest VS dogfood, running build.cmd -ninja from root:

cl : Command line error D8016 : '/std:c++latest' and '/std:c11' command-line options are incompatible
cl : Command line warning D9025 : overriding '/TP' with '/TC'

while building src/coreclr/pal/src/libunwind

[jkotas]

cc @jkoritzinsky

[tommcdon]

@agocke @trylek this affects the dev inner loop

[trylek]

If this is due to some recent changes in the C++ compiler installed with the new VS, @janvorli may need to take a look to figure out the proper way to adjust the cmake files to be backward-compatible.

[agocke]

Reading the docs here, can we just disable stdc for libunwind? Since it specifies C++latest I assume it doesn't care about the C compatibility version

[agocke]

Well that seemed to work, but it looks like there's something wrong with the DAC: I get the error The application to execute does not exist: 'C:/Users/andy/Code/runtime/artifacts/bin/coreclr/Linux.x64.Debug/DacTableGen/DacTableGen.dll'

It looks like it's trying to consume a Linux DAC even though this is building on Windows. Do we build a cross dac by default?

[tommcdon]

It looks like it's trying to consume a Linux DAC even though this is building on Windows. Do we build a cross dac by default?

Adding @sdmaclea

[sdmaclea]

Yes this looks like it is due to building the linux cross dac for Windows. I'll take a look.

[vitek-karas]

This is still not fully fixed for me - on latest master it fails with:

     'C:/ProgramData/chocolatey/bin/ninja.exe' '-C' 'F:/dotnet/runtime2/artifacts/obj/native/net6.0-windows-Debug-x64' '-t' 'cleandead'

    failed with:

ninja : error : build.ninja:945: multiple rules generate System.IO.Compression.Native/System.IO.Compression.Native.lib [-w dupbuild=err] [F:\dotnet\runtime2\src\libraries\Native\build-native.proj]

[jkoritzinsky]

cc: @VSadov since he most recently changed how we build System.IO.Compression.Native

[VSadov]

The library was renamed on windows. It used to be clrcompression and now it is System.IO.Compression.Native - same as on Unix. That was basically the change.
I wonder if this could be related to the rename somehow?

[akoeplinger]

The System.IO.Compression.Native.lib issue was apparently fixed by 5761dd4

[vitek-karas]

Now fails with a different error:

 -- Build files have been written to: F:/dotnet/runtime2/artifacts/obj/native/net6.0-windows-Debug-x64
  [1/94] Building C object System.IO.Compression.Native\CMakeFiles\System.IO.Compression.Native.dir\F_\dotnet\runtime2\src\libraries\Native\AnyOS\brotli\common\context.c.obj
cl : Command line error D9025: overriding '/W0' with '/W3' [F:\dotnet\runtime2\src\libraries\Native\build-native.proj]
cl : Command line error D9025: overriding '/MDd' with '/MTd' [F:\dotnet\runtime2\src\libraries\Native\build-native.proj]

[akoeplinger]

Yeah I see the same here.