diff options
| author | Thang Tran <trankimthang279@gmail.com> | 2019-02-06 02:58:43 +0100 |
|---|---|---|
| committer | Richard Lau <riclau@uk.ibm.com> | 2019-02-15 06:34:36 -0500 |
| commit | 128170f5c6d26fea3c46ecb1f523e12a5a8093e1 (patch) | |
| tree | 0f082dca1fff53a7aebc0384213c61ddac8991e5 | |
| parent | 51d20b6a8f695947a700e7fed8f0e5b33b79cefb (diff) | |
| download | android-node-v8-128170f5c6d26fea3c46ecb1f523e12a5a8093e1.tar.gz android-node-v8-128170f5c6d26fea3c46ecb1f523e12a5a8093e1.tar.bz2 android-node-v8-128170f5c6d26fea3c46ecb1f523e12a5a8093e1.zip | |
build: fixed clang's warning when building openssl
clang doesn't seem to support 'Wno-old-style-declaration', this
is a work-around.
Fixes: https://github.com/nodejs/node/issues/25550
Refs: https://github.com/nodejs/node-v0.x-archive/issues/4186
PR-URL: https://github.com/nodejs/node/pull/25954
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
| -rw-r--r-- | deps/openssl/openssl_common.gypi | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/deps/openssl/openssl_common.gypi b/deps/openssl/openssl_common.gypi index 75002e87c2..c4b95c20ae 100644 --- a/deps/openssl/openssl_common.gypi +++ b/deps/openssl/openssl_common.gypi @@ -57,14 +57,17 @@ ], }, { # linux and others - 'cflags': ['-Wno-missing-field-initializers', - ## TODO: check gcc_version>=4.3 - '-Wno-old-style-declaration'], + 'cflags': ['-Wno-missing-field-initializers',], 'defines': [ 'OPENSSLDIR="/etc/ssl"', 'ENGINESDIR="/dev/null"', 'TERMIOS', ], + 'conditions': [ + [ 'llvm_version==0', { + 'cflags': ['-Wno-old-style-declaration',], + }], + ], }], ] } |