-
Notifications
You must be signed in to change notification settings - Fork 445
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
Simple console app requires ASP.NET packages #1958
Comments
I notice in the generated project.assets.json:
But I guess that doesn't help with where it's coming from. I don't have enough context on how this works to figure that out. |
@dsplaisted can you take a look? |
This is expected. The SDK will download the runtime pack for ASP.NET Core in case the app references any packages (or other projects) that depend on it. If there isn't anything that depends on it, it will not include it in the publish output. |
cc @jkotas I see. It is non intuitive that I need to add an ASP.NET feed, but given that in production this will be all on the same nuget feed, I guess it is not something that should matter to customers. |
It will matter to customers that publish self-contained apps. They will need to download a lot more stuff than what's required... |
Why did this happen in the first place? |
IIRC this is for this reason: dotnet/sdk#10077
Targeting packs are normally installed with the .NET SDK, so you won't observe this behavior with them, but I think the same explanation applies to runtime packs, which are always resolved via NuGet. @dsplaisted please correct me if I'm wrong. |
Yes. We could avoid these downloads, but it would required a new feature in NuGet. We've discussed it before but I couldn't find it documented anywhere, so I've filed this issue to capture it: NuGet/Home#9937 |
A way to work around this was checked in a while back. See dotnet/sdk#25358 for setting |
I should not need ASP.NET nor should publishing it include the ASP.NET runtime.
The text was updated successfully, but these errors were encountered: