diff options
author | Stefan Metzmacher <metze@samba.org> | 2015-01-07 09:16:57 +0100 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2015-01-08 23:38:07 +0100 |
commit | 46fdd6221e9c64b459e0d05e6d02d89deb8dafb5 (patch) | |
tree | 9dc0353cc68a3c277e33c6de19ce8f4be5ec1f57 /buildtools/wafsamba/wscript | |
parent | 52eb8854ff8535577c9282f8e4133875fbb0ed33 (diff) | |
download | samba-46fdd6221e9c64b459e0d05e6d02d89deb8dafb5.tar.gz |
wafsamba: improve -fvisibility=hidden, we should check it together this WERROR_CFLAGS
GCC ignores -fvisibility=hidden with a warning instead of failing
om some platforms (e.g. Solaris).
Bug: https://bugzilla.samba.org/show_bug.cgi?id=11031
Based on a patch from Tom Schulz <schulz@adi.com>.
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
Diffstat (limited to 'buildtools/wafsamba/wscript')
-rwxr-xr-x | buildtools/wafsamba/wscript | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/buildtools/wafsamba/wscript b/buildtools/wafsamba/wscript index 0a1cd1ce3ea..c49b20e1153 100755 --- a/buildtools/wafsamba/wscript +++ b/buildtools/wafsamba/wscript @@ -341,7 +341,7 @@ def configure(conf): else: conf.env.HAVE_LD_VERSION_SCRIPT = False - if conf.CHECK_CFLAGS('-fvisibility=hidden'): + if conf.CHECK_CFLAGS(['-fvisibility=hidden'] + conf.env.WERROR_CFLAGS): conf.env.VISIBILITY_CFLAGS = '-fvisibility=hidden' conf.CHECK_CODE('''int main(void) { return 0; } __attribute__((visibility("default"))) void vis_foo2(void) {}''', |