win, build: fix building addons on Windows

bzoz · refack · commit f23139f1f691 · 2019-01-10T10:04:20.000-05:00
Building addons would fail because addon-verify.js dependencies from tools\doc where not installed. This fixes this issue by installing those dependencies if addons are to be built. Fixes: nodejs#25096 PR-URL: nodejs#25108 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
diff --git a/vcbuild.bat b/vcbuild.bat
@@ -446,8 +446,8 @@ if errorlevel 1 goto exit
 
 
 :install-doctools
-REM only install if building doc OR testing doctool
-if not defined doc (
+REM only install if building doc OR testing doctool OR building addons
+if not defined doc if not defined build_addons (
   echo.%test_args% | findstr doctool 1>nul
   if errorlevel 1 goto :skip-install-doctools
 )

Original file line number	Diff line number	Diff line change
`@@ -446,8 +446,8 @@ if errorlevel 1 goto exit`
`446`	`446`
`447`	`447`
`448`	`448`	`:install-doctools`
`449`		`-REM only install if building doc OR testing doctool`
`450`		`-if not defined doc (`
	`449`	`+REM only install if building doc OR testing doctool OR building addons`
	`450`	`+if not defined doc if not defined build_addons (`
`451`	`451`	`echo.%test_args% \| findstr doctool 1>nul`
`452`	`452`	`if errorlevel 1 goto :skip-install-doctools`
`453`	`453`	`)`