Skip to content

Warn that NuGet.exe needs .NET Framework MSBuild#7261

Open
zivkan wants to merge 2 commits intodevfrom
dev-zivkan-nuget.exe-dotnet-core-msbuild
Open

Warn that NuGet.exe needs .NET Framework MSBuild#7261
zivkan wants to merge 2 commits intodevfrom
dev-zivkan-nuget.exe-dotnet-core-msbuild

Conversation

@zivkan
Copy link
Copy Markdown
Member

@zivkan zivkan commented Apr 7, 2026

Bug

Fixes: NuGet/Home#14844

Description

Detect when a .NET SDK directory was provided and report an error. NuGet.exe runs on the .NET Framework runtime and therefore can't use the .NET (Core) runtime MSBuild.

PR Checklist

  • Meaningful title, helpful description and a linked NuGet/Home issue
  • Added tests
  • Link to an issue or pull request to update docs if this PR changes settings, environment variables, new feature, etc. N/A

@zivkan zivkan requested a review from a team as a code owner April 7, 2026 23:19
@zivkan zivkan requested review from Nigusu-Allehu and jebriede April 7, 2026 23:19
@@ -1,4 +1,4 @@
<?xml version="1.0" encoding="utf-8"?>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Wondering if these no change changes are co related to #7256 (comment)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe the issue is in our repo, and the local bot was fixing it.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'm sure it's related to how copilot modified the xlf files. I reverted the changes, then ran a build to let the xlf tools update the files, and it's all good now.

Nigusu-Allehu
Nigusu-Allehu previously approved these changes Apr 8, 2026
Copy link
Copy Markdown
Member

@Nigusu-Allehu Nigusu-Allehu left a comment

Choose a reason for hiding this comment

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

outside of the invisible character change in the xlfs, this looks good to me

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.

nuget.exe restore -MSBuildPath crashes when pointing it to .NET SDK directory

2 participants