-
Notifications
You must be signed in to change notification settings - Fork 30k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tons of unknown warning option '-Wno-old-style-declaration' when compiling openssl #25550
Comments
There's even an old comment about that in
configure.py already knows the gcc version (search for |
|
config.gypi is created when you run configure/configure.py. |
It would probably be better to check for |
Urgh, that's what I mean: The comment in openssl_common.gypi is about |
I'm working on a fix for this issue. |
@ibc Could you please try this command and share the output to us echo "__clang__ __GNUC__ __GNUC_MINOR__ __GNUC_PATCHLEVEL__ __clang_major__ __clang_minor__ __clang_patchlevel__" | /usr/bin/clang++-7 -E -P -x c++ - |
@bnoordhuis I believed the config.gypi already has the clang version via ['variables']['llvm_version']. Or did i misunderstand something?
|
Hi @thangktran:
|
it seems to be a problem with clang itself. testaccount@testmachine:~$ echo "__clang__ __GNUC__ __GNUC_MINOR__ __GNUC_PATCHLEVEL__ __clang_major__ __clang_minor__ __clang_patchlevel__" | /usr/bin/clang-7 -E -P -Wno-old-style-declaration -
warning: unknown warning option '-Wno-old-style-declaration'; did you mean '-Wno-out-of-line-declaration'? [-Wunknown-warning-option]
1 4 2 1 7 0 0
1 warning generated. This error appears in clang-5 clang-6 and clang-7. |
clang doesn't seem to support 'Wno-old-style-declaration', this is a work-around. Fixes: nodejs#25550 Refs: nodejs/node-v0.x-archive#4186
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>
Thanks. |
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>
Version: v10.13.0
Platform: Linux e4b2ecc253f9 4.9.125-linuxkit x86_64 GNU/Linux (Docker in Mac)
clang version: 7.0.0 (tags/RELEASE_700/final)
When compiling openssl tons of warning are printed like this:
It does not happen when using GCC compiler, but just with clang, which seems to not support
this option the cause.
The text was updated successfully, but these errors were encountered: