Commit c07ba96
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
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -961,6 +961,10 @@ | |||
| 961 | 961 | 'OTHER_LDFLAGS': [ '-Wl,-rpath,@loader_path', ], | |
| 962 | 962 | }, | |
| 963 | 963 | }], | |
| 964 | + # Skip cctest while building shared lib node for Windows | ||
| 965 | + [ 'OS=="win" and node_shared=="true"', { | ||
| 966 | + 'type': 'none', | ||
| 967 | + }], | ||
| 964 | 968 | ], | |
| 965 | 969 | } | |
| 966 | 970 | ], # end targets | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -474,6 +474,7 @@ if errorlevel 1 goto exit | |||
| 474 | 474 | if "%test_args%"=="" goto test-v8 | |
| 475 | 475 | if "%config%"=="Debug" set test_args=--mode=debug %test_args% | |
| 476 | 476 | if "%config%"=="Release" set test_args=--mode=release %test_args% | |
| 477 | + if not exist %config%\cctest.exe goto run-test-py | ||
| 477 | 478 | echo running 'cctest %cctest_args%' | |
| 478 | 479 | "%config%\cctest" %cctest_args% | |
| 479 | 480 | REM when building a static library there's no binary to run tests | |
0 commit comments