Skip to content

Update the nuget version to the latest and update STJ to the in-product one that's being used already in most of the build here #9083

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

Open
wants to merge 4 commits into
base: release/9.0.1xx
Choose a base branch
from

Conversation

marcpopMSFT
Copy link
Member

I tried just updating to 8.0.5 but I found that much of the build was already implicitly targeting 9.0.3. So instead, I just retargeted to that. Not sure if this will have negative impact on SDK or VS though so will need to consult on that.

I could just move the edge assembly as that was the only one flagged by CG or I could get nuget to fix their reference.

…ct one that's being used already in most of the build here
@marcpopMSFT marcpopMSFT requested a review from rainersigwald June 9, 2025 17:07
@marcpopMSFT marcpopMSFT requested a review from a team as a code owner June 9, 2025 17:07
@marcpopMSFT
Copy link
Member Author

@dotnet/source-build-internal another case where I have to add a bunch of items to SBRP? Annoying to have to do this with each nuget update.

See https://aka.ms/dotnet/prebuilts for guidance on what pre-builts are and how to eliminate them.
Package IDs are:
NuGet.Common.6.12.4
NuGet.Configuration.6.12.4
NuGet.Credentials.6.12.4
NuGet.Frameworks.6.12.4
NuGet.Packaging.6.12.4
NuGet.Protocol.6.12.4
NuGet.Versioning.6.12.4
System.IO.Pipelines.9.0.3
System.Text.Encodings.Web.9.0.3
System.Text.Json.9.0.3

@ellahathaway
Copy link
Member

ellahathaway commented Jun 11, 2025

@dotnet/source-build-internal another case where I have to add a bunch of items to SBRP? Annoying to have to do this with each nuget update.

Yes, the NuGet packages need to be added to SBRP. I understand the frustration. Unfortunately, the VMR isn't configured for the templating repo to depend on the nuget-client repo, which means that there's no live version of NuGet to flow in. No live flow means that these packages must be defined in SBRP.

@MichaelSimons
Copy link
Member

Yes, the NuGet packages need to be added to SBRP. I understand the frustration. Unfortunately, the VMR isn't configured for the templating repo to depend on the nuget-client repo, which means that there's no live version of NuGet to flow in. No live flow means that these packages must be defined in SBRP.

Would it be feasible for templating to depend on the live version for source-build? e.g. can a V.D.xml dependency be declared on NuGet but not have a darc subscription for it?

@ellahathaway
Copy link
Member

Yes, the NuGet packages need to be added to SBRP. I understand the frustration. Unfortunately, the VMR isn't configured for the templating repo to depend on the nuget-client repo, which means that there's no live version of NuGet to flow in. No live flow means that these packages must be defined in SBRP.

Would it be feasible for templating to depend on the live version for source-build? e.g. can a V.D.xml dependency be declared on NuGet but not have a darc subscription for it?

This would change the build order, so as long as nuget-client doesn't require packages from repos downstream of templating, I think it would be okay. I can test in the VMR.

@MichaelSimons
Copy link
Member

This would change the build order, so as long as nuget-client doesn't require packages from repos downstream of templating, I think it would be okay. I can test in the VMR.

I am not proposing that a project dependency be added in SB. If the V.D.xml dependency is added and the project is built before nuget, it will pickup the n-1 version. I am assuming these don't get bundled because of the use of SBRP packages today so this should be fine.

@ellahathaway
Copy link
Member

ellahathaway commented Jun 11, 2025

This would change the build order, so as long as nuget-client doesn't require packages from repos downstream of templating, I think it would be okay. I can test in the VMR.

I am not proposing that a project dependency be added in SB. If the V.D.xml dependency is added and the project is built before nuget, it will pickup the n-1 version. I am assuming these don't get bundled because of the use of SBRP packages today so this should be fine.

I completely forgot about PSB artifacts for a moment. You're right, this should work. I'll update version details

@MichaelSimons
Copy link
Member

I completely forgot about PSB artifacts for a moment. You're right, this should work. I'll update version details

That change really should be validated in the context of the VMR as well. You could easily do that in the 9.0 branch of the VMR by opening a PR with only that change.

@marcpopMSFT
Copy link
Member Author

The only thing I'm not sure if is if updating V.D.xml will impact the flow to sdk. The SDK repo ends up on the same sha nuget but it's an unstable version rather than stable version. I don't know if we'll end up with downgrade errors (as they are technically preview versions so "older") when trying to flow to sdk or if CPM will handle that.

@ellahathaway were you going to try updating V.D.xml or want me to do it? I don't want to add the extra subscription as that adds to our build graph for releases but I can update that manually after release day for future nuget msrcs

@ellahathaway
Copy link
Member

@ellahathaway were you going to try updating V.D.xml or want me to do it?

Testing in dotnet/dotnet#1135

@ellahathaway
Copy link
Member

ellahathaway commented Jun 12, 2025

@marcpopMSFT - the VMR validation passed. These are the changes you need. The prebuilts will continue to pop up in repo-level build (eg in this current PR), so you'll also have to add exclusions to the prebuilt baseline for those. Let me know if you'd me to port these changes over for you. Thanks :)

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.

4 participants