From: Lasse Collin <lasse.collin@tukaani.org>
Date: Wed, 1 Jul 2009 09:21:24 +0000 (+0300)
Subject: Avoid visibility related compiler warnings on Windows.
X-Git-Tag: v4.999.9beta~60
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5cc99db5bae8633f85559e5cdaef4cd905a4ee9c;p=thirdparty%2Fxz.git

Avoid visibility related compiler warnings on Windows.
---

diff --git a/configure.ac b/configure.ac
index b1961eca..2f3af3d0 100644
--- a/configure.ac
+++ b/configure.ac
@@ -554,21 +554,19 @@ lc_CPUCORES
 # If using GCC, set some additional AM_CFLAGS:
 ###############################################################################
 
-if test "x$GCC" = xyes ; then
+if test "$GCC" = yes ; then
 	echo
 	echo "GCC extensions:"
+fi
 
-	# Avoid checking for visibility support on Windows, because the test
-	# may succeed even though visibility isn't supported. Windows has
-	# a different way to export only the required symbols from the
-	# libraries.
-	if test "$is_w32" = no; then
-		gl_VISIBILITY
-		if test -n "$CFLAG_VISIBILITY" ; then
-			AM_CFLAGS="$AM_CFLAGS $CFLAG_VISIBILITY"
-		fi
-	fi
+# Always do the visibility check but don't set AM_CFLAGS on Windows.
+# This way things get set properly even on Windows.
+gl_VISIBILITY
+if test -n "$CFLAG_VISIBILITY" && test "$is_w32" = no; then
+	AM_CFLAGS="$AM_CFLAGS $CFLAG_VISIBILITY"
+fi
 
+if test "$GCC" = yes ; then
 	# Enable as much warnings as possible. These commented warnings won't
 	# work for this package though:
 	#   * -Wunreachable-code breaks several assert(0) cases, which are