diff options
author | Refael Ackermann <refack@gmail.com> | 2018-09-30 20:39:54 +0300 |
---|---|---|
committer | Michaël Zasso <targos@protonmail.com> | 2018-10-04 12:02:38 +0300 |
commit | 9800fd7aaa7226fff3f21edd7f77f563b9a69753 (patch) | |
tree | bacd71b07b07cca60c19a7871ac8f17e31a0d799 /node.gypi | |
parent | 51f0060765a400c77553c29936baaf044b85c29b (diff) |
build,deps: refactor and fix v8.gyp
* Make inspector.gypi and v8_external_snapshot.gypi includible targets.
* Make `v8_dump_build_config` an action
* Better separate `js2c` and `natives_blob`
* process action outputs as sources
* trigger v8.gyp:postmortem-metadata from v8.gyp
PR-URL: https://github.com/nodejs/node/pull/23182
Refs: https://github.com/nodejs/node/pull/23156
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Diffstat (limited to 'node.gypi')
-rw-r--r-- | node.gypi | 21 |
1 files changed, 6 insertions, 15 deletions
diff --git a/node.gypi b/node.gypi index fe133039d3d..55ba20e2ecb 100644 --- a/node.gypi +++ b/node.gypi @@ -122,21 +122,12 @@ [ 'node_no_browser_globals=="true"', { 'defines': [ 'NODE_NO_BROWSER_GLOBALS' ], } ], - [ 'node_use_bundled_v8=="true" and v8_postmortem_support=="true"', { - 'conditions': [ - # -force_load is not applicable for the static library - [ 'force_load=="true"', { - 'xcode_settings': { - 'OTHER_LDFLAGS': [ - '-Wl,-force_load,<(v8_base)', - ], - }, - }], - # when building with GN, the v8_monolith target already includes postmortem metadata - [ 'build_v8_with_gn=="false"', { - 'dependencies': [ 'deps/v8/gypfiles/v8.gyp:postmortem-metadata' ], - }], - ], + [ 'node_use_bundled_v8=="true" and v8_postmortem_support=="true" and force_load=="true"', { + 'xcode_settings': { + 'OTHER_LDFLAGS': [ + '-Wl,-force_load,<(v8_base)', + ], + }, }], [ 'node_shared_zlib=="false"', { 'dependencies': [ 'deps/zlib/zlib.gyp:zlib' ], |