Skip to content

Conversation

@jkoritzinsky
Copy link
Member

Add support for only building the current vertical's installers in pass 1 (and building the hosting bundle in pass 2) when building aspnetcore in the VMR.

Also, change how we download "known" assets to use the same mechanism as dotnet/sdk so the VMR can point to local artifacts.

This PR will require a reactive change in the VMR that adds https://github.com/dotnet/sdk/blob/b5a9089d86579d5a0f600478634db4ac1044cc75/src/SourceBuild/content/repo-projects/sdk.proj#L37 to the aspnetcore.proj project (which I'll prep now).

This should set up enough infra that we can turn on building installers on Windows for aspnetcore in the VMR and that (I hope) most if not all changes required for the hosting bundle can be done within the VMR.

Contributes to dotnet/source-build#4692 (Windows)

Add support for only building the current vertical's installers (and building the hosting bundle in pass 2) when building aspnetcore in the VMR.
… for the VMR just like we have for dotnet/sdk
@jkoritzinsky jkoritzinsky requested review from a team and wtgodbe as code owners November 6, 2024 18:59
@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Nov 6, 2024
jkoritzinsky added a commit to jkoritzinsky/dotnet-sdk that referenced this pull request Nov 6, 2024
…s from the local artifacts assets path

Required to consume dotnet/aspnetcore#58820 in the VMR
Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

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

The Build.props changes are a bit hard to review but aside from that, LGTM

Copy link
Member

@wtgodbe wtgodbe left a comment

Choose a reason for hiding this comment

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

Looks reasonable, thanks!

@jkoritzinsky jkoritzinsky merged commit 5f935fe into dotnet:main Nov 7, 2024
27 checks passed
@jkoritzinsky jkoritzinsky deleted the no-hosting-bundle-vmr-pass-1 branch November 7, 2024 19:24
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0-preview1 milestone Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants