Skip to content

Conversation

EgorBo
Copy link
Member

@EgorBo EgorBo commented Jul 30, 2025

Fixes #118093

@Copilot Copilot AI review requested due to automatic review settings July 30, 2025 12:24
@EgorBo EgorBo requested review from BrzVlad, janvorli and kg as code owners July 30, 2025 12:24
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a build issue with the -msbuild build for the CLR subset by addressing a CMake compatibility problem with Visual Studio generators. The change ensures that the EXCLUDE_FROM_ALL property is only set for non-Visual Studio generators to prevent build failures.

Key Changes:

  • Adds conditional logic to prevent setting EXCLUDE_FROM_ALL property when using Visual Studio generators

@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jul 30, 2025
@EgorBo
Copy link
Member Author

EgorBo commented Jul 30, 2025

PTAL @jkoritzinsky @kotlarmilos

@kotlarmilos kotlarmilos self-requested a review July 31, 2025 07:40
Copy link
Member

@kotlarmilos kotlarmilos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Sorry, I missed cc on the issue.

@EgorBo EgorBo merged commit abc4c65 into dotnet:main Jul 31, 2025
94 of 96 checks passed
@EgorBo EgorBo deleted the fix-msbuild-build branch July 31, 2025 09:18
@github-actions github-actions bot locked and limited conversation to collaborators Aug 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Clr -msbuild" build is broken by clrinterpreter

3 participants