<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/3.26.0">
</HEAD>
<BODY>
On Fri, 2010-03-19 at 13:53 -0700, Alan Coopersmith wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
Gaetan Nadon wrote:
&gt; The server sticks -fvisibility compiler flag in the pkg Cflags which
&gt; does not help while 99% of pkg-config files on my distro are compiler
&gt; flags free.

Which is probably a bug in itself - I end up removing @symbol_visibility@
from xorg-server.pc.in when building our packages, since we support both
gcc &amp; Sun Studio, and they take different flags for that option.

</PRE>
</BLOCKQUOTE>
<BR>
I tend to agree, providing compiler flags such as this one is pushing the &quot;convenience&quot; envelop. This flag should be specified in the driver module, but it is significant work. There is no facility to specify a portable compiler flag. In this particular case, it could be supplied through a server macro and included in each driver makefile. <BR>
<BR>
If we are willing to make the assumption that no other modules will pass a compiler flag in pkgconfig, then we can have pre-processor/compiler flags separation in apps and libs. <BR>
<BR>
As a side note, it looks like the server config handles the SUN compilers.<BR>
<BR>
                AC_CHECK_DECL([__SUNPRO_C], [SUNCC=&quot;yes&quot;], [SUNCC=&quot;no&quot;])<BR>
                if test x$SUNCC = xyes; then<BR>
                &nbsp;&nbsp;&nbsp; VISIBILITY_CFLAGS=&quot;-xldscope=hidden&quot;<BR>
                else<BR>
                &nbsp;&nbsp;&nbsp; have_visibility=no<BR>
                fi<BR>
<BR>
</BODY>
</HTML>