diff options
author | Daniel Bevenius <daniel.bevenius@gmail.com> | 2017-08-23 09:52:59 +0200 |
---|---|---|
committer | Daniel Bevenius <daniel.bevenius@gmail.com> | 2017-11-16 09:22:13 +0100 |
commit | b021403b2afdab0f5077b55aa27286c187119acd (patch) | |
tree | 7f43c95b7872ff024670a22c205b8b087f57880a /node.gypi | |
parent | a0b1e2eca684a21e87dd8ae1d5becb4c85e4ddd4 (diff) | |
download | android-node-v8-b021403b2afdab0f5077b55aa27286c187119acd.tar.gz android-node-v8-b021403b2afdab0f5077b55aa27286c187119acd.tar.bz2 android-node-v8-b021403b2afdab0f5077b55aa27286c187119acd.zip |
test: --enable-static linked executable
The motivation for this commit is to enable projects embedding Node.js
and building with --enable-static to be able to run the test suite and
linter.
Currently when building with --enable-static no node executable
will be created which means that the tests (apart from the cctest) and
linter cannot be run.
This is currently a work in progress and works on MacOS but I need to
run the CI, and manually on different environments to verify that it
works as expected.
PR-URL: https://github.com/nodejs/node/pull/14986
Refs: https://github.com/nodejs/node/issues/14158
Refs: https://github.com/nodejs/node/pull/14892
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Diffstat (limited to 'node.gypi')
-rw-r--r-- | node.gypi | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -78,7 +78,7 @@ 'src/node_lttng.cc' ], } ], - [ 'node_use_etw=="true"', { + [ 'node_use_etw=="true" and node_target_type!="static_library"', { 'defines': [ 'HAVE_ETW=1' ], 'dependencies': [ 'node_etw' ], 'sources': [ @@ -90,7 +90,7 @@ 'tools/msvs/genfiles/node_etw_provider.rc', ] } ], - [ 'node_use_perfctr=="true"', { + [ 'node_use_perfctr=="true" and node_target_type!="static_library"', { 'defines': [ 'HAVE_PERFCTR=1' ], 'dependencies': [ 'node_perfctr' ], 'sources': [ |