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