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 <[email protected]> Reviewed-By: Daniel Bevenius <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Feb 28, 2019 · f408d78 · f408d78
1 parent 14cf22f
commit f408d78
Showing 1 changed file with 6 additions and 3 deletions.
diff --git 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',],
+        }],
+      ],
     }],
   ]
 }