diff options
author | Kyle Fuller <kyle@fuller.li> | 2018-08-08 17:04:01 +0100 |
---|---|---|
committer | Anna Henningsen <anna@addaleax.net> | 2018-08-24 01:12:14 +0200 |
commit | 3ba54e4cc844f3c7ba62ad8622e709fc51f0b6bf (patch) | |
tree | 7e0fa3890412f8146a803acef6820b43631211b7 /common.gypi | |
parent | 4963a04b304e5417f5ba8f769ae8040c18fd018d (diff) | |
download | android-node-v8-3ba54e4cc844f3c7ba62ad8622e709fc51f0b6bf.tar.gz android-node-v8-3ba54e4cc844f3c7ba62ad8622e709fc51f0b6bf.tar.bz2 android-node-v8-3ba54e4cc844f3c7ba62ad8622e709fc51f0b6bf.zip |
build: Don't set `-fno-threadsafe-statics` on macOS
This flag is not set on other platforms so it can produce inconsistent
behaviour across platforms. For example, if you build an async node
add-on which uses statics you can get race conditions due to static
not supporting threads if the node add-on inherits from the Node
common.gypi config. It is not disabled on other platforms such as
Linux, it is not disabled by default in Xcode or clang.
This setting has been there since the initial commit that introduces
`common.gypi` and thus has been there since the start, it doesn't seem
to be have added for any particular reason other than to potentially
match the Xcode defaults at the time.
PR-URL: https://github.com/nodejs/node/pull/22198
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'common.gypi')
-rw-r--r-- | common.gypi | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/common.gypi b/common.gypi index ec877db990..db9401ffa1 100644 --- a/common.gypi +++ b/common.gypi @@ -447,7 +447,6 @@ 'GCC_ENABLE_CPP_EXCEPTIONS': 'NO', # -fno-exceptions 'GCC_ENABLE_CPP_RTTI': 'NO', # -fno-rtti 'GCC_ENABLE_PASCAL_STRINGS': 'NO', # No -mpascal-strings - 'GCC_THREADSAFE_STATICS': 'NO', # -fno-threadsafe-statics 'PREBINDING': 'NO', # No -Wl,-prebind 'MACOSX_DEPLOYMENT_TARGET': '10.7', # -mmacosx-version-min=10.7 'USE_HEADERMAP': 'NO', |