Update NugetProjectModelVersion and NugetPackagingVersion from 6.2.2 to 6.2.4

[LoopedBard3]

Update NugetProjectModelVersion and NugetPackagingVersion from 6.2.2 to 6.2.4

[ghost]

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

Issue Details

---

Update NugetProjectModelVersion and NugetPackagingVersion from 6.2.2 to 6.2.4

Author:	LoopedBard3
Assignees:	LoopedBard3
Labels:	area-Infrastructure-libraries
Milestone:	-

[LoopedBard3]

Test build in the performance repo: https://dev.azure.com/dnceng/internal/_build/results?buildId=2210578&view=results

[carlossanlop]

Assuming there are no CI issues, LGTM.

[ViktorHofer]

Curious, why?

[ViktorHofer]

@MichaelSimons @mthalman I might be wrong, but shouldn't this result in a new prebuild in the source build leg as that package version isn't available in SBRP?

[mthalman]

@MichaelSimons @mthalman I might be wrong, but shouldn't this result in a new prebuild in the source build leg as that package version isn't available in SBRP?

I don't see a source-build leg even running in the pipeline.

[ViktorHofer]

At least this one runs:

runtime-dev-innerloop (Build linux-x64 Release PortableSourceBuild)

[mthalman]

Ah, this needs to be updated:

runtime/eng/Version.Details.xml

Lines 382 to 384 in c06d77a

	<Dependency Name="Nuget.ProjectModel" Version="6.2.2">
	<Uri>https://github.com/NuGet/NuGet.Client</Uri>
	<Sha>027ca8b8ef4b4dc94995f87b9c441d2bcf742c1d</Sha>

This isn't showing up in prebuilt reporting because all NuGet packages are excluded from that:

runtime/eng/SourceBuildPrebuiltBaseline.xml

Lines 13 to 14 in c06d77a

	<!-- Will be removed once https://github.com/NuGet/Home/issues/11059 is resolved -->
	<UsagePattern IdentityGlob="Nuget.*/*" />

[LoopedBard3]

Ah, this needs to be updated:

runtime/eng/Version.Details.xml

Lines 382 to 384 in c06d77a

	<Dependency Name="Nuget.ProjectModel" Version="6.2.2">
	<Uri>https://github.com/NuGet/NuGet.Client</Uri>
	<Sha>027ca8b8ef4b4dc94995f87b9c441d2bcf742c1d</Sha>

This isn't showing up in prebuilt reporting because all NuGet packages are excluded from that:

runtime/eng/SourceBuildPrebuiltBaseline.xml

Lines 13 to 14 in c06d77a

	<!-- Will be removed once https://github.com/NuGet/Home/issues/11059 is resolved -->
	<UsagePattern IdentityGlob="Nuget.*/*" />

I updated the spot linked. Do you know if there is anywhere else that needs to be updated with these values?

[LoopedBard3]

It looks like the "Build windows-x86 release MSBuild_CMake" job is failing for multiple PR's based on looking at the pipeline, due to "Failed to generate native component build project!". I am not finding an already opened tracking issue for this, is there a way to create a tracking issue or is opening a bug report the best way? Also, is this PR good to merge? Thanks!

[ViktorHofer]

This isn't showing up in prebuilt reporting because all NuGet packages are excluded from that:

Got it, thanks. But do we still need an SBRP package for it?

[mthalman]

Got it, thanks. But do we still need an SBRP package for it?

No, it wouldn't rely on SBRP in full source-build. Because it's listed as a dependency in Version.Details.xml, it will make use of the package from previously source-built artifacts.

[ViktorHofer]

Cool, thanks. @LoopedBard3 can you please explain why this change is necessary?

[agocke]

/backport to release/7.0

[github-actions]

Started backporting to release/7.0: https://github.com/dotnet/runtime/actions/runs/6828957092

[github-actions]

@agocke backporting to release/7.0 failed, the patch most likely resulted in conflicts:

$ git am --3way --ignore-whitespace --keep-non-patch changes.patch

Applying: Update Nuget Package Versions.
Using index info to reconstruct a base tree...
M	eng/Versions.props
Falling back to patching base and 3-way merge...
Auto-merging eng/Versions.props
CONFLICT (content): Merge conflict in eng/Versions.props
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Update Nuget Package Versions.
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

[github-actions]

@agocke an error occurred while backporting to release/7.0, please check the run log for details!

Error: git am failed, most likely due to a merge conflict.