On Wed, Feb 4, 2009 at 2:01 PM, Paulo César Pereira de Andrade
<[email protected]> wrote:
>  Another trivial patch.

Looking at another one of these, I see small nit.

configure.ac:
XORG_CFLAGS="$XORG_CFLAGS $CWARNFLAGS"

Makefile.am:
AM_CFLAGS = $(XORG_CFLAGS)

It would be nicer if it was just

Makefile.am:
AM_CFLAGS = $(XORG_CFLAGS) $(CWARNFLAGS)

Then I can actually manipulate the warning flags when I build instead
of just using what configure came up with. I.e.,

make CWARNFLAGS="-Werror"

if I'm so inclined. Not a big deal, though.

--
Dan
_______________________________________________
xorg mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/xorg

Reply via email to