1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

build: fixed clang's warning when building openssl

Browse Source 
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>
This commit is contained in:
Thang Tran 2019-02-06 02:58:43 +01:00 committed by Richard Lau
parent 51d20b6a8f
commit 128170f5c6
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
deps/openssl/openssl_common.gypi vendored
View File

@ -57,14 +57,17 @@
], ],
}, { }, {
# linux and others # linux and others
'cflags': ['-Wno-missing-field-initializers', 'cflags': ['-Wno-missing-field-initializers',],
## TODO: check gcc_version>=4.3
'-Wno-old-style-declaration'],
'defines': [ 'defines': [
'OPENSSLDIR="/etc/ssl"', 'OPENSSLDIR="/etc/ssl"',
'ENGINESDIR="/dev/null"', 'ENGINESDIR="/dev/null"',
'TERMIOS', 'TERMIOS',
], ],
'conditions': [
[ 'llvm_version==0', {
'cflags': ['-Wno-old-style-declaration',],
}],
],
}], }],
] ]
} }