◐ Shell
clean mode source ↗

[v8.x-backport] build: skip cctest on Windows shared lib build by BethGriggs · Pull Request #25758 · nodejs/node

@BethGriggs changed the title [v8.x-staging] build: skip cctest on Windows shared lib build [v8.x-backport] build: skip cctest on Windows shared lib build

Jan 28, 2019

richardlau

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>

PR-URL: nodejs#21228
Reviewed-By: Refael Ackermann <refack@gmail.com>

BethGriggs pushed a commit that referenced this pull request

Mar 20, 2019
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>