Hey guys,
Just wanted to know what the requirements are for Building CoreFX on Windows with VS 2017, and maybe suggest an update to the docs here https://github.com/dotnet/corefx/blob/master/Documentation/building/windows-instructions.md.
From a quick test, it seems that I need the Desktop development with C++ workload, but also the VC++ 2015.3 v140 toolset (x86,x64) component.
I tried installing only these individual packages but CMake wouldn't find the compiler (No CMAKE_CXX_COMPILER could be found):
Regarding the DIA SDK, it seems the Desktop development with C++ workload installs it in C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\DIA SDK and build.cmd finds it without any problem.
Thanks!
I would have expected https://github.com/dotnet/corefx/pull/16710 to fix these issues.
cc @tannergooding
The docs could definitely be updated.
I haven't attempted to find the minimum set of components required, but I believe it should work if you have the following workloads
In any case, VS2017 only works if you run the build script from the Developer Command Prompt.
I will send out a PR later tonight that updates the docs. I will also determine what the minimal install required is.
Feel free to make me a reviewer. I'll test the minimal install in my computer.
Ok, so for the Workloads install, looks like the following is minimal:
For Individual Components Install, looks like the following is minimal:
The build instructions for CoreCLR were already partially updated, but it requires a workaround which I didn't document and one or more components not listed above (still trying to determine which component that is).
I will send a PR for both CoreFX and CoreCLR to list the minimum dependencies here after I finish validating the CoreCLR requirements.
Actually, looks like CoreCLR isn't working at all anymore (WRT building on VS2017), due to the recent build tools upgrade: https://github.com/dotnet/coreclr/commit/8f23fca0ff57c97707d4f6b5068e8044d2dcc13c
I'll send the PR out for CoreFX and will continue investigating the CoreCLR issue separately
Most helpful comment
The docs could definitely be updated.
I haven't attempted to find the minimum set of components required, but I believe it should work if you have the following workloads
In any case, VS2017 only works if you run the build script from the Developer Command Prompt.
I will send out a PR later tonight that updates the docs. I will also determine what the minimal install required is.