Skip to content

Commit 0ec2e1d

Browse files
committed
build: skip cctest on Windows shared lib build
cctest depends on some internal APIs which don't declare `__declspec(dllexport)` and causes build failure when building node as shared lib on Windows. Since we already have good test coverage in static lib, we decide to skip the cctest in shared lib build on Windows. Signed-off-by: Yihong Wang <yh.wang@ibm.com>
1 parent f5a2167 commit 0ec2e1d

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

node.gyp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -989,6 +989,10 @@
989989
['OS=="solaris"', {
990990
'ldflags': [ '-I<(SHARED_INTERMEDIATE_DIR)' ]
991991
}],
992+
# Skip cctest while building shared lib node for Windows
993+
[ 'OS=="win" and node_shared=="true"', {
994+
'type': 'none',
995+
}],
992996
],
993997
}
994998
], # end targets

vcbuild.bat

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -542,6 +542,7 @@ if "%test_args%"=="" goto test-v8
542542
if "%config%"=="Debug" set test_args=--mode=debug %test_args%
543543
if "%config%"=="Release" set test_args=--mode=release %test_args%
544544
if defined no_cctest echo Skipping cctest because no-cctest was specified && goto run-test-py
545+
if not exist %config%\cctest.exe goto run-test-py
545546
echo running 'cctest %cctest_args%'
546547
"%config%\cctest" %cctest_args%
547548
:run-test-py

0 commit comments

Comments
 (0)