Skip to content

Commit cd5c76c

Browse files
stduhpfaduh95
authored andcommitted
tools: fix vcbuild test when path contain spaces
PR-URL: #56481 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
1 parent da6cb8e commit cd5c76c

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

vcbuild.bat

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ if defined package set stage_package=1
184184
:: assign path to node_exe
185185
set "node_exe=%config%\node.exe"
186186
set "node_gyp_exe="%node_exe%" deps\npm\node_modules\node-gyp\bin\node-gyp"
187-
set "npm_exe="%~dp0%node_exe%" %~dp0deps\npm\bin\npm-cli.js"
187+
set "npm_exe="%node_exe%" deps\npm\bin\npm-cli.js"
188188
if "%target_env%"=="vs2022" set "node_gyp_exe=%node_gyp_exe% --msvs_version=2022"
189189

190190
:: skip building if the only argument received was lint
@@ -608,9 +608,7 @@ if not defined doc if not defined build_addons (
608608
)
609609
if exist "tools\doc\node_modules\unified\package.json" goto skip-install-doctools
610610
SETLOCAL
611-
cd tools\doc
612-
%npm_exe% ci
613-
cd ..\..
611+
%npm_exe% --prefix tools\doc ci
614612
if errorlevel 1 goto exit
615613
ENDLOCAL
616614
:skip-install-doctools
@@ -753,9 +751,7 @@ goto lint-js-build
753751

754752
:lint-js-build
755753
if not defined lint_js_build if not defined lint_js if not defined lint_js_fix goto lint-md-build
756-
cd tools\eslint
757-
%npm_exe% ci
758-
cd ..\..
754+
%npm_exe% --prefix tools\eslint ci
759755

760756
:lint-js
761757
if not defined lint_js goto lint-js-fix
@@ -773,9 +769,7 @@ goto lint-md-build
773769

774770
:lint-md-build
775771
if not defined lint_md if not defined format_md goto lint-md
776-
cd tools\lint-md
777-
%npm_exe% ci
778-
cd ..\..
772+
%npm_exe% --prefix tools\lint-md ci
779773

780774
:lint-md
781775
if not defined lint_md goto format-md

0 commit comments

Comments
 (0)