build: work around bug in MSBuild v16.10.0

targos · targos · commit fdba113a12d0 · 2021-06-05T10:33:19.000+02:00
This allows to build Node.js at the (temporary) cost of longer build times. Refs: #38872 Refs: https://github.com/dotnet/msbuild/releases/tag/v16.10.0 Refs: dotnet/msbuild#6465 Refs: dotnet/msbuild#6373 PR-URL: #38873 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
diff --git a/vcbuild.bat b/vcbuild.bat
@@ -372,6 +372,8 @@ if "%target%"=="Build" (
   if defined cctest set target="Build"
 )
 if "%target%"=="node" if exist "%config%\cctest.exe" del "%config%\cctest.exe"
+@rem TODO(targos): Remove next line after MSBuild 16.10.1 is released.
+if "%target%"=="node" set target="Build"
 if defined msbuild_args set "extra_msbuild_args=%extra_msbuild_args% %msbuild_args%"
 @rem Setup env variables to use multiprocessor build
 set UseMultiToolTask=True

Original file line number	Diff line number	Diff line change
`@@ -372,6 +372,8 @@ if "%target%"=="Build" (`
`372`	`372`	`if defined cctest set target="Build"`
`373`	`373`	`)`
`374`	`374`	`if "%target%"=="node" if exist "%config%\cctest.exe" del "%config%\cctest.exe"`
	`375`	`+@rem TODO(targos): Remove next line after MSBuild 16.10.1 is released.`
	`376`	`+if "%target%"=="node" set target="Build"`
`375`	`377`	`if defined msbuild_args set "extra_msbuild_args=%extra_msbuild_args% %msbuild_args%"`
`376`	`378`	`@rem Setup env variables to use multiprocessor build`
`377`	`379`	`set UseMultiToolTask=True`