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

Build the "in build" crossgen2 into a pack that we can use within the VMR #110676

Merged
merged 25 commits into from
Jan 10, 2025

Conversation

jkoritzinsky
Copy link
Member

@jkoritzinsky jkoritzinsky commented Dec 13, 2024

For the same reasons that we need an "in build" crossgen2 in dotnet/runtime, we need one in the VMR for cross-builds of downstream repos (like aspnetcore).

Opening as draft until I can get enough pieces together to validate the end-to-end experience. This will need changes in aspnetcore, windowsdesktop, sdk, and the orchestration for dotnet/runtime (to put the host crossgen2 package in a place where it won't accidentally get off the machine or cause conflicts in asset selection).

Contributes to dotnet/source-build#3793

Copy link
Contributor

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

…ary and it breaks the new crossgen2 host package
…gen2 tasks in the SDK to discover the target OS from the source-build target RID
@jkoritzinsky jkoritzinsky marked this pull request as ready for review December 18, 2024 23:21
@jkoritzinsky jkoritzinsky requested review from a team and ivdiazsa December 18, 2024 23:23
eng/DotNetBuild.props Outdated Show resolved Hide resolved
eng/Signing.props Outdated Show resolved Hide resolved
eng/Signing.props Outdated Show resolved Hide resolved
@ViktorHofer
Copy link
Member

/ba-g stopped hearing from agent ...

@ViktorHofer ViktorHofer merged commit 71db8b2 into dotnet:main Jan 10, 2025
153 of 155 checks passed
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.

3 participants