From 5cc99db5bae8633f85559e5cdaef4cd905a4ee9c Mon Sep 17 00:00:00 2001 From: Lasse Collin Date: Wed, 1 Jul 2009 12:21:24 +0300 Subject: [PATCH] Avoid visibility related compiler warnings on Windows. --- configure.ac | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) 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