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