Mesa (master): Don't use -fvisibilty=hidden on cygwin
Jon TURNEY
jturney at kemper.freedesktop.org
Tue Jun 28 16:16:26 UTC 2011
Module: Mesa
Branch: master
Commit: db78643182dc39ed592dd8c2e5fc7c8eeb7316a1
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=db78643182dc39ed592dd8c2e5fc7c8eeb7316a1
Author: Jon TURNEY <jon.turney at dronecode.org.uk>
Date: Tue Apr 26 11:56:02 2011 +0100
Don't use -fvisibilty=hidden on cygwin
All it's going to do is generate lots and lots and lots of
'warning: visibility attribute not supported in this configuration; ignored'
warnings
Signed-off-by: Jon TURNEY <jon.turney at dronecode.org.uk>
Reviewed-by: Dan Nicholson <dbn.lists at gmail.com>
---
configure.ac | 9 +++++++++
1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/configure.ac b/configure.ac
index 74357a2..7fbba7d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -189,6 +189,15 @@ if test "x$GXX" = xyes; then
CXXFLAGS="$CXXFLAGS -fno-strict-aliasing"
fi
+dnl even if the compiler appears to support it, using visibility attributes isn't
+dnl going to do anything useful currently on cygwin apart from emit lots of warnings
+case "$host_os" in
+cygwin*)
+ VISIBILITY_CFLAGS=""
+ VISIBILITY_CXXFLAGS=""
+ ;;
+esac
+
AC_SUBST([VISIBILITY_CFLAGS])
AC_SUBST([VISIBILITY_CXXFLAGS])
More information about the mesa-commit
mailing list