Skip to content

Resolve build failure when SSMS is installed#6575

Merged
mitchelsellers merged 1 commit intodnnsoftware:developfrom
bdukes:nuget-vswhere
Jun 13, 2025
Merged

Resolve build failure when SSMS is installed#6575
mitchelsellers merged 1 commit intodnnsoftware:developfrom
bdukes:nuget-vswhere

Conversation

@bdukes
Copy link
Copy Markdown
Contributor

@bdukes bdukes commented Jun 5, 2025

Summary

After installing SQL Server Management Studio (SSMS) 21, I noticed that I started getting errors during the build of the platform. SSMS 21 started using the Visual Studio Installer to manage installations and updates, so it appears that NuGet is finding and using its version of MSBuild instead of Visual Studio's.

This PR uses vswhere to find the latest Visual Studio version and pass that to NuGet.exe

See also NuGet/Home#14349

@bdukes bdukes added this to the 10.0.2 milestone Jun 5, 2025
Copy link
Copy Markdown
Contributor

@valadas valadas left a comment

Choose a reason for hiding this comment

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

Good catch!

@valadas
Copy link
Copy Markdown
Contributor

valadas commented Jun 5, 2025

Looks like CI did not like it though...

@bdukes
Copy link
Copy Markdown
Contributor Author

bdukes commented Jun 5, 2025

Weird. I adjusted it a smidge and it looks to be working.

@bdukes bdukes force-pushed the nuget-vswhere branch 2 times, most recently from 6a420d4 to 4bac38d Compare June 12, 2025 13:42
@mitchelsellers mitchelsellers merged commit 058cff2 into dnnsoftware:develop Jun 13, 2025
2 checks passed
@mitchelsellers mitchelsellers deleted the nuget-vswhere branch June 13, 2025 17:22
@valadas valadas modified the milestones: 10.0.2, 10.1.0 Jul 30, 2025
corbob added a commit to corbob/Chocolatey.Cake.Recipe that referenced this pull request Dec 4, 2025
This commit updates the logic for detecting the MSBuild tool. Our builds
encountered this issue: NuGet/Home#14349.

While investigating it, it was found that dnnsoftware encounted a
similar issue and fixed it here:
dnnsoftware/Dnn.Platform#6575. This commit
updates the methods we use to call VSWhere to identify the MSBuild to
return based on actually having an MSBuild.exe file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants