<!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.32.2">
</HEAD>
<BODY>
On Mon, 2011-10-17 at 21:21 +0200, Julien Cristau wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
On Mon, Oct 17, 2011 at 11:37:08 -0700, Jeremy Huddleston wrote:

&gt; This should force the server to have -fno-strict-aliasing even once it
&gt; is removed from the warning flags.
&gt; 
&gt; See: <A HREF="https://bugs.freedesktop.org/show_bug.cgi?id=31238">https://bugs.freedesktop.org/show_bug.cgi?id=31238</A>
&gt; 
&gt; Signed-off-by: Jeremy Huddleston &lt;<A HREF="mailto:jeremyhu@apple.com">jeremyhu@apple.com</A>&gt;
&gt; ---
&gt;  configure.ac |    6 ++++++
&gt;  1 files changed, 6 insertions(+), 0 deletions(-)
&gt; 
&gt; diff --git xorg-server-1.11.1/configure.ac xorg-server-1.11.1/configure.ac
&gt; index 4656a83..538aa88 100644
&gt; --- xorg-server-1.11.1/configure.ac
&gt; +++ xorg-server-1.11.1/configure.ac
&gt; @@ -87,6 +87,12 @@ XORG_PROG_RAWCPP
&gt;  # easier overrides at build time.
&gt;  XSERVER_CFLAGS='$(CWARNFLAGS)'
&gt;  
&gt; +dnl Explicitly add -fno-strict-aliasing since this option should disappear
&gt; +dnl from util-macros CWARNFLAGS
&gt; +if  test &quot;x$GCC&quot; = xyes ; then
&gt; +    XSERVER_CFLAGS=&quot;$XSERVER_CFLAGS -fno-strict-aliasing&quot;
&gt; +fi
&gt; +
&gt;  dnl Check for dtrace program (needed to build Xserver dtrace probes)
&gt;  dnl Also checks for &lt;sys/sdt.h&gt;, since some Linux distros have an 
&gt;  dnl ISDN trace program named dtrace

ack in principle, although the comment is going to look slightly odd
if/when CWARNFLAGS actually drops the flag.
</PRE>
</BLOCKQUOTE>
The most likely follow-on step is to add another variable which contains only warning flags as CWARNFLAGS cannot be changed in a way that would affect previously released tarballs. There is no easy or elegant way to fix the current situation.<BR>
<BR>
The CWARNFLAGS variable would be deprecated and left alone for backward compatibility. All actively maintained modules would replace it with the new variable.
<BLOCKQUOTE TYPE=CITE>
<PRE>

Cheers,
Julien
</PRE>
</BLOCKQUOTE>
<BR>
</BODY>
</HTML>