Commit bf9fdceef63507da889a22eb1a583d2b6376f5ea

Gaetan Nadon 2010-12-08T20:37:26

config: remove incorrectly implemented visibility compiler option 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>

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
diff --git a/configure.ac b/configure.ac
index 4fa9ea7..b982839 100644
--- a/configure.ac
+++ b/configure.ac
@@ -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])