Skip to content

(#282) Allow building with the latest SSMS installed.#283

Draft
corbob wants to merge 2 commits intochocolatey:developfrom
corbob:build-with-ssms
Draft

(#282) Allow building with the latest SSMS installed.#283
corbob wants to merge 2 commits intochocolatey:developfrom
corbob:build-with-ssms

Conversation

@corbob
Copy link
Copy Markdown
Member

@corbob corbob commented Dec 4, 2025

Description Of Changes

Update the logic to find an MSBuild that actually has an MSBuild?

Motivation and Context

Being able to build things while the latest SSMS is installed.

Testing

For any project using this recipe, apply a similar diff to this:

diff --git a/recipe.cake b/recipe.cake
index bc029d1f..30269b28 100644
--- a/recipe.cake
+++ b/recipe.cake
@@ -1,4 +1,5 @@
-#load nuget:?package=Chocolatey.Cake.Recipe&version=0.30.1
+#load nuget:?package=Chocolatey.Cake.Recipe&version=0.30.1&Include=/**/version.cake
+#load local:?path=C:/code/chocolatey.cake.recipe/chocolatey.cake.recipe/Content/*.cake
 #tool nuget:?package=WiX&version=3.11.2

Run the build script.

Operating Systems Testing

Windows 11

Change Types Made

  • Bug fix (non-breaking change).
  • Feature / Enhancement (non-breaking change).
  • Breaking change (fix or feature that could cause existing functionality to change).
  • Documentation changes.
  • PowerShell code changes.

Change Checklist

  • Requires a change to the documentation.
  • Documentation has been updated.
  • Tests to cover my changes, have been added.
  • All new and existing tests passed?
  • PowerShell code changes: PowerShell v3 compatibility checked?

Related Issue

@gep13
Copy link
Copy Markdown
Member

gep13 commented Dec 4, 2025

@corbob our futureselves are going to want/need more words in the commit message than that! 😄

@corbob
Copy link
Copy Markdown
Member Author

corbob commented Dec 4, 2025

@corbob our futureselves are going to want/need more words in the commit message than that! 😄

@gep13 I know that 🤪 Current me was just getting things up while I was waiting for a packer build to finish! But tomorrow me is a future me that will likely want more information, so thank you very much for the nudge today while I have some of the information in my head 👍

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.
I've found a better way to do local development on the recipe, so this
updates the Readme with that information.
@corbob corbob marked this pull request as draft December 4, 2025 22:13
@corbob
Copy link
Copy Markdown
Member Author

corbob commented Dec 4, 2025

This is draft while I work through any details needed here... It's also possible that this is no longer an issue once Visual Studio 2026 is installed.

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.

Unable to build projects with the latest Sql Server Management Studio installed

2 participants