config: remove incorrectly implemented visibility compiler option
authorGaetan Nadon <memsize@videotron.ca>
Thu, 9 Dec 2010 01:37:26 +0000 (20:37 -0500)
committerGaetan Nadon <memsize@videotron.ca>
Sat, 18 Dec 2010 16:19:56 +0000 (11:19 -0500)
CFLAGS is a user variable which should never be set by the configuration.
It allows the user to alter the configuration compiler options.

The visibility is only set for GNU compiler, leaving libraries built
with other compilers with the wrong visibility.

All other xorg libraries set visibilty using _X_EXPORT or _X_HIDDEN.

For the time being, all the symbols will have the default visibility
which does not break anything.

Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
configure.ac

index 4fa9ea7..b982839 100644 (file)
@@ -65,10 +65,6 @@ if test "x$CC_FOR_BUILD" = x; then
     fi
 fi
 
-if  test "x$GCC" = xyes ; then
-  CFLAGS="$CFLAGS -fvisibility=hidden"
-fi
-
 # Obtain protocols headers include directives
 PKG_CHECK_MODULES([X11], [xproto kbproto >= 1.0.5])