[PATCH util-macros v2 3/6] Use XORG_TESTSET_CFLAG in XORG_STRICT_OPTION

Jeremy Huddleston jeremyhu at apple.com
Thu Nov 3 14:58:06 PDT 2011


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<jeremyhu at apple.com>
>> ---
>>  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-        alan.coopersmith at oracle.com
> 	 Oracle Solaris Platform Engineering: X Window System
> 



More information about the xorg-devel mailing list