Skip to content
Browse files

build,win: propagate error codes in vcbuild

Don't exit vcbuild with error code 0 when cctest fails.

PR-URL: #30724
Refs: nodejs/build#1996
Reviewed-By: Rod Vagg <>
Reviewed-By: Sam Roberts <>
Reviewed-By: Rich Trott <>
  • Loading branch information
joaocgreis authored and MylesBorins committed Nov 30, 2019
1 parent 6a35c38 commit 3b096c2bc473030a7af01d6d2bcdb4286d184bd6
Showing with 4 additions and 1 deletion.
  1. +4 −1 vcbuild.bat
@@ -68,6 +68,7 @@ set cctest=
set openssl_no_asm=
set doc=
set extra_msbuild_args=
set exit_code=0

if "%1"=="" goto args-done
@@ -629,9 +630,11 @@ if defined no_cctest echo Skipping cctest because no-cctest was specified && got
if not exist "%config%\cctest.exe" echo cctest.exe not found. Run "vcbuild test" or "vcbuild cctest" to build it. && goto run-test-py
echo running 'cctest %cctest_args%'
"%config%\cctest" %cctest_args%
if %errorlevel% neq 0 set exit_code=%errorlevel%
echo running 'python tools\ %test_args%'
python tools\ %test_args%
if %errorlevel% neq 0 set exit_code=%errorlevel%
goto test-v8

@@ -714,7 +717,7 @@ echo vcbuild.bat no-cctest : skip building cctest.exe
goto exit

goto :EOF
exit /b %exit_code%

rem ***************

0 comments on commit 3b096c2

Please sign in to comment.
You can’t perform that action at this time.