Skip to content
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

Propagate RuntimeConfiguration for cross-subset builds #107049

Merged
merged 2 commits into from
Aug 28, 2024

Conversation

elinor-fung
Copy link
Member

This addresses issues building cross-subset with clr with mixed configurations.

For example, singlefilehost is built as part of the clr subset, but the host subset copies it, embeds the DAC (when necessary) and packages it. When the clr and host configuration didn't match, this resulted in the host build looking for singlefilehost in the wrong location.

@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Aug 27, 2024
@elinor-fung elinor-fung added area-Infrastructure and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Aug 27, 2024
@elinor-fung elinor-fung requested a review from a team August 27, 2024 17:53
Copy link
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

eng/Subsets.props Outdated Show resolved Hide resolved
Co-authored-by: Jeremy Koritzinsky <jkoritzinsky@gmail.com>
@elinor-fung elinor-fung changed the title Propagate CoreCLRConfiguration for cross-subset builds Propagate RuntimeConfiguration for cross-subset builds Aug 28, 2024
@elinor-fung
Copy link
Member Author

/ba-g timeouts in legs unaffected by this change

@elinor-fung elinor-fung merged commit 063ae05 into dotnet:main Aug 28, 2024
147 of 149 checks passed
@elinor-fung elinor-fung deleted the propagate-clr-config branch August 28, 2024 22:38
jtschuster pushed a commit to jtschuster/runtime that referenced this pull request Sep 17, 2024
This addresses issues building cross-subset with clr (or mono) with mixed configurations.

For example, `singlefilehost` is built as part of the `clr` subset, but the `host` subset copies it, embeds the DAC (when necessary) and packages it. When the clr and host configuration didn't match, this resulted in the host build looking for `singlefilehost` in the wrong location.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants