From ffc1b453cfcd3f880f7520cf05cf233755310a94 Mon Sep 17 00:00:00 2001
From: EWSoftware <Eric@EWoodruff.us>
Date: Fri, 30 Dec 2022 11:23:40 -0800
Subject: [PATCH] NuGet SHFB package property file change

Changed the NuGet SHFB package build properties file to use `BuildingInsideVisualStudio` instead of an empty SHFBROOT value to determine when to define SHFBROOT for command line builds.  Fixes #955 and should also resolve #920 but only in cases where projects use a package version containing this change (a version later the 2022.10.15.1).  See the workaround in #955 for earlier packages.
---
 NuGet/EWSoftware.SHFB.props                  | 2 +-
 TestCaseProject/Doc/TestCaseProject.shfbproj | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/NuGet/EWSoftware.SHFB.props b/NuGet/EWSoftware.SHFB.props
index 887c5ebf..f33f00a6 100644
--- a/NuGet/EWSoftware.SHFB.props
+++ b/NuGet/EWSoftware.SHFB.props
@@ -1,5 +1,5 @@
 <Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
   <PropertyGroup>
-    <SHFBROOT Condition="$(SHFBROOT) == ''">$(MSBuildThisFileDirectory)..\tools\</SHFBROOT >
+    <SHFBROOT Condition="$(BuildingInsideVisualStudio) != 'true'">$(MSBuildThisFileDirectory)..\tools\</SHFBROOT>
   </PropertyGroup>
 </Project>
diff --git a/TestCaseProject/Doc/TestCaseProject.shfbproj b/TestCaseProject/Doc/TestCaseProject.shfbproj
index 6ae50b29..06c40375 100644
--- a/TestCaseProject/Doc/TestCaseProject.shfbproj
+++ b/TestCaseProject/Doc/TestCaseProject.shfbproj
@@ -398,8 +398,8 @@
   <!-- Uncomment this to use the nuget packages when SHFB is not installed locally, e.g. in the CI build pipeline.
        Update the package versions as required.
   <ItemGroup>
-    <PackageReference Include="EWSoftware.SHFB" Version="2019.9.15" />
-    <PackageReference Include="EWSoftware.SHFB.NETFramework" Version="4.8.0" />
+    <PackageReference Include="EWSoftware.SHFB" Version="2022.10.15.0" />
+    <PackageReference Include="EWSoftware.SHFB.NETFramework" Version="4.8.0.2" />
   </ItemGroup> -->
   <!-- Import the common build targets during NuGet restore because before the packages are installed, $(SHFBROOT) is not set yet -->
   <Import Project="$(MSBuildToolsPath)\Microsoft.Common.targets" Condition="'$(MSBuildRestoreSessionId)' != ''" />