-
Notifications
You must be signed in to change notification settings - Fork 4.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Official build and CI builds failed with CMake Warning due to build agents updated VS #31888
Comments
This is showing up now on the runtime-live-build pipeline. |
Hello, I'm trying to build the latest master locally on windows 10, Visual Studio VS 2019, 16.4.1 and I'm getting this error:
This seems to be the same error as #30845, but I have the code from the PRs in this pull request disabling warning as errors. I don't have CMake installed, it's unclear whether this is still necessary. The instructions about building under windows, which where once here, seem to have been removed: dotnet/corefx@81c91fa#diff-4b7d5829731d454157a0e13d9d46da42 |
@robertpi - I believe the updated build instructions for the consolidated runtime repo are at https://github.com/dotnet/runtime/blob/master/docs/workflow/README.md and related. I'm not sure what PR you are referring to. I fixed the issue for the runtime-live-build pipeline with Please however note that this fix is specific to lab testing - it only modifies the yml scripts to pass around an extra parameter so it doesn't affect local builds. AFAIK our standard developer guidance is that building the dotnet/runtime repo requires CMake 3.15.5 or newer. |
It was just that CMake was missing. It builds fine with the latest cmake installed. Might be nice to have a link from the building docs to the requirements page, that's what confused me. Should I open a PR for that. |
@robertpi we already document that in the following way:
|
Official builds are currently hitting: #30845 because the build agents got updated to VS2019 16.4 and CMake is 3.15.1, we need 3.15.5 or later.
I will disable WarnAsError for Windows in the official builds shortly. We should revert that when this is fixed.
cc: @dotnet/runtime-infrastructure @MattGal
The text was updated successfully, but these errors were encountered: