Skip to content

Comments

.NET SDK release/dnup branch build uses dotnetup to download core .NET runtimes#8

Closed
nagilson wants to merge 4 commits intonagilson-dnup-install-runtimes-implfrom
nagilson-sdk-uses-dnup-for-core-runtimes
Closed

.NET SDK release/dnup branch build uses dotnetup to download core .NET runtimes#8
nagilson wants to merge 4 commits intonagilson-dnup-install-runtimes-implfrom
nagilson-sdk-uses-dnup-for-core-runtimes

Conversation

@nagilson
Copy link
Owner

@nagilson nagilson commented Jan 27, 2026

Building on top of dotnet#52649, we want to be a first party dogfooder to QA our product, evaluate its functionality as a user, and gain confidence in its ability to be publicly consumed elsewhere. This is not going to apply to main or any official branch currently releasing because we still need approval for that pending other work items.

We will eventually install nightly SDK and Runtimes with this as well, but this is a good stop gap for now. To make that work we need to download a dotnetup executable, because we can't build dotnetup without the SDK. Since we don't have that acquisition experience of dotnetup itself done, and it doesn't yet install preview SDKs, I think this is good for now.

Note this will fail quite often until dotnet#52696 is merged which fixes a bug in dotnetup.

InitializeDotnetCli installs the .NET 10 SDK
EnsureDotnetupBuilt checks if dotnetup.exe exists, builds it if not
InstallDotNetSharedFramework uses dotnetup to install runtimes 6.0, 7.0, 8.0, 9.0

- I considered containing the dotnetup exe itself or acquiring it but that would require sign checks to be secure.
…mpl' into nagilson-sdk-uses-dnup-for-core-runtimes
We also get rid of the deletion of pdbs - it's an unncessary step.
Also, we don't fall back to the install script - we want to fail if dotnetup fails so we know it's failing. It should never fail. It if fails we want to find out why and fix it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant