diff options
author | Yihong Wang <yh.wang@ibm.com> | 2018-03-07 12:25:26 -0800 |
---|---|---|
committer | Yihong Wang <yh.wang@ibm.com> | 2018-03-12 16:56:04 -0700 |
commit | ffd618bd5cde77e19ab6458eaf454c4df71dd638 (patch) | |
tree | ce3ac6ceb25fcb8818764856d02f5407137bcb4d /node.gyp | |
parent | 0acdd8402614ce39a95a798b7e2898972332c4c7 (diff) | |
download | android-node-v8-ffd618bd5cde77e19ab6458eaf454c4df71dd638.tar.gz android-node-v8-ffd618bd5cde77e19ab6458eaf454c4df71dd638.tar.bz2 android-node-v8-ffd618bd5cde77e19ab6458eaf454c4df71dd638.zip |
test: shared lib build doesn't handle SIGPIPE
For shared lib build, we leave the signal handling for embedding users.
In these two test cases:
- `parallel/test-process-external-stdio-close-spawn`
- `parallel/test-process-external-stdio-close`
The pipe is used for stdout and is destroied before child process uses
it for logging. So the node executble that uses shared lib build
receives SIGPIPE and the child process ends.
This change ignores the SIGPIPE in node_main.cc for shared lib case.
Refs: https://github.com/nodejs/node/issues/18535
Signed-off-by: Yihong Wang <yh.wang@ibm.com>
PR-URL: https://github.com/nodejs/node/pull/19211
Refs: https://github.com/nodejs/node/issues/18535
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Diffstat (limited to 'node.gyp')
-rw-r--r-- | node.gyp | 20 |
1 files changed, 3 insertions, 17 deletions
@@ -203,6 +203,9 @@ 'sources': [ 'src/node_main.cc' ], + 'includes': [ + 'node.gypi' + ], 'include_dirs': [ 'src', 'deps/v8/include', @@ -220,9 +223,6 @@ }], [ 'node_intermediate_lib_type=="static_library" and ' 'node_shared=="false"', { - 'includes': [ - 'node.gypi' - ], 'xcode_settings': { 'OTHER_LDFLAGS': [ '-Wl,-force_load,<(PRODUCT_DIR)/<(STATIC_LIB_PREFIX)' @@ -469,22 +469,8 @@ ], }], ], - 'defines!': [ - 'NODE_PLATFORM="win"', - ], - 'defines': [ - 'FD_SETSIZE=1024', - # we need to use node's preferred "win32" rather than gyp's preferred "win" - 'NODE_PLATFORM="win32"', - # Stop <windows.h> from defining macros that conflict with - # std::min() and std::max(). We don't use <windows.h> (much) - # but we still inherit it from uv.h. - 'NOMINMAX', - '_UNICODE=1', - ], 'libraries': [ '-lpsapi.lib' ] }, { # POSIX - 'defines': [ '__POSIX__' ], 'sources': [ 'src/backtrace_posix.cc' ], }], [ 'node_use_etw=="true"', { |