Skip to content

Commit f408d78

Browse files
thangktranrvagg
authored andcommitted
build: fixed clang's warning when building openssl
clang doesn't seem to support 'Wno-old-style-declaration', this is a work-around. Fixes: #25550 Refs: nodejs/node-v0.x-archive#4186 PR-URL: #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>
1 parent 14cf22f commit f408d78

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

deps/openssl/openssl_common.gypi

+6-3
Original file line numberDiff line numberDiff line change
@@ -57,14 +57,17 @@
5757
],
5858
}, {
5959
# linux and others
60-
'cflags': ['-Wno-missing-field-initializers',
61-
## TODO: check gcc_version>=4.3
62-
'-Wno-old-style-declaration'],
60+
'cflags': ['-Wno-missing-field-initializers',],
6361
'defines': [
6462
'OPENSSLDIR="/etc/ssl"',
6563
'ENGINESDIR="/dev/null"',
6664
'TERMIOS',
6765
],
66+
'conditions': [
67+
[ 'llvm_version==0', {
68+
'cflags': ['-Wno-old-style-declaration',],
69+
}],
70+
],
6871
}],
6972
]
7073
}

0 commit comments

Comments
 (0)