Skip to content

Break infinite restore loop in VS#15503

Merged
nohwnd merged 1 commit into
microsoft:mainfrom
nohwnd:fix-vsix-restore
Mar 17, 2026
Merged

Break infinite restore loop in VS#15503
nohwnd merged 1 commit into
microsoft:mainfrom
nohwnd:fix-vsix-restore

Conversation

@nohwnd
Copy link
Copy Markdown
Member

@nohwnd nohwnd commented Mar 17, 2026

Description

Workaround for dotnet/arcade#16228

Copilot AI review requested due to automatic review settings March 17, 2026 10:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Implements a workaround for dotnet/arcade#16228 by preventing Visual Studio from entering an infinite NuGet restore loop caused by a property name collision for the Microsoft.VSSDK.BuildTools version.

Changes:

  • Renames the Microsoft.VSSDK.BuildTools version MSBuild property to a collision-resistant name (_MicrosoftVSSDKBuildToolsVersion_).
  • Updates Microsoft.TestPlatform.csproj to consume the new property name.
  • Adds a repo-wide PackageReference Update to pin/override Microsoft.VSSDK.BuildTools to the intended version.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/package/Microsoft.TestPlatform/Microsoft.TestPlatform.csproj Switches the Microsoft.VSSDK.BuildTools version to the new underscore-delimited property.
eng/Versions.props Introduces _MicrosoftVSSDKBuildToolsVersion_ and documents the restore-loop workaround.
Directory.Build.targets Adds a repo-wide PackageReference Update to ensure the pinned build tools version is applied.

Comment thread src/package/Microsoft.TestPlatform/Microsoft.TestPlatform.csproj
Comment thread eng/Versions.props
Comment thread Directory.Build.targets
@nohwnd nohwnd merged commit ad2ae67 into microsoft:main Mar 17, 2026
4 of 7 checks passed
@nohwnd
Copy link
Copy Markdown
Member Author

nohwnd commented Mar 17, 2026

force merged, only applies to VS (most often) and it built on server as well

This was referenced May 26, 2026
This was referenced May 29, 2026
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.

2 participants