Ah, I squashed that into the wrong commit.

This line should be in commit 4:

> +AC_REQUIRE([XORG_COMPILER_FLAGS])

On Nov 3, 2011, at 12:55, Alan Coopersmith wrote:

> On 11/03/11 01:19, Jeremy Huddleston wrote:
>> Signed-off-by: Jeremy Huddleston<[email protected]>
>> ---
>>  xorg-macros.m4.in |   29 ++++++++++-------------------
>>  1 files changed, 10 insertions(+), 19 deletions(-)
>> 
>> diff --git a/xorg-macros.m4.in b/xorg-macros.m4.in
>> index 1ece326..123bdaa 100644
>> --- a/xorg-macros.m4.in
>> +++ b/xorg-macros.m4.in
>> @@ -1468,31 +1468,22 @@ AC_SUBST(CWARNFLAGS)
>>  # Starting in 1.14.0 also exports $STRICT_CFLAGS for use in other tests or
>>  # when strict compilation is unconditionally desired.
>>  AC_DEFUN([XORG_STRICT_OPTION], [
>> -# If the module's configure.ac calls AC_PROG_CC later on, CC gets set to C89
>> -AC_REQUIRE([AC_PROG_CC_C99])
>> -AC_REQUIRE([XORG_COMPILER_BRAND])
>>  AC_REQUIRE([XORG_CWARNFLAGS])
>> +AC_REQUIRE([XORG_COMPILER_FLAGS])
> 
> Was that supposed to be in this patch, since it requires a macro not defined 
> until patch 4?
> 
> -- 
>       -Alan Coopersmith-        [email protected]
>        Oracle Solaris Platform Engineering: X Window System
> 

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to