镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Commit c07ba96

Browse files
yhwangBethGriggs
authored andcommitted
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> Backport-PR-URL: #25758 PR-URL: #21228 Reviewed-By: Refael Ackermann <refack@gmail.com>
1 parent 6352288 commit c07ba96

2 files changed

Lines changed: 5 additions & 0 deletions

File tree

‎node.gyp‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -961,6 +961,10 @@
961961
'OTHER_LDFLAGS': [ '-Wl,-rpath,@loader_path', ],
962962
},
963963
}],
964+
# Skip cctest while building shared lib node for Windows
965+
[ 'OS=="win" and node_shared=="true"', {
966+
'type': 'none',
967+
}],
964968
],
965969
}
966970
], # end targets

‎vcbuild.bat‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -474,6 +474,7 @@ if errorlevel 1 goto exit
474474
if "%test_args%"=="" goto test-v8
475475
if "%config%"=="Debug" set test_args=--mode=debug %test_args%
476476
if "%config%"=="Release" set test_args=--mode=release %test_args%
477+
if not exist %config%\cctest.exe goto run-test-py
477478
echo running 'cctest %cctest_args%'
478479
"%config%\cctest" %cctest_args%
479480
REM when building a static library there's no binary to run tests

0 commit comments

Comments
 (0)