[PATCH RESEND util-macros] XORG_PROG_RAWCPP: use AC_LANG_SOURCE to prevent warnings

Alan Coopersmith alan.coopersmith at oracle.com
Sat Jun 4 20:36:06 PDT 2011


On 06/ 4/11 05:00 AM, Gaetan Nadon wrote:
> The source for the AC_LANG_CONFTEST should be passed to AC_LANG_SOURCE
> to satisfy the more stringent testing done by autoconf 2.68.
> It is available in autoconf 2.60 where AC_LANG_DEFINES_PROVIDED is not.

And you're right, it doesn't change what is generated in the resulting configure
script with 2.68, so I guess my worry about it changing the test was for
nothing.

>  xorg-macros.m4.in |    6 ++----
>  1 files changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/xorg-macros.m4.in b/xorg-macros.m4.in
> index a21cf7e..d2ad154 100644
> --- a/xorg-macros.m4.in
> +++ b/xorg-macros.m4.in
> @@ -64,8 +64,7 @@ AC_PATH_PROGS(RAWCPP, [cpp], [${CPP}],
>  # which is not the best choice for supporting other OS'es, but covers most
>  # of the ones we need for now.
>  AC_MSG_CHECKING([if $RAWCPP requires -undef])
> -AC_LANG_CONFTEST([Does cpp redefine unix ?
> -		  AC_LANG_DEFINES_PROVIDED])
> +AC_LANG_CONFTEST([AC_LANG_SOURCE([[Does cpp redefine unix ?]])])
>  if test `${RAWCPP} < conftest.$ac_ext | grep -c 'unix'` -eq 1 ; then
>  	AC_MSG_RESULT([no])
>  else
> @@ -83,8 +82,7 @@ fi
>  rm -f conftest.$ac_ext
>  
>  AC_MSG_CHECKING([if $RAWCPP requires -traditional])
> -AC_LANG_CONFTEST([Does cpp preserve   "whitespace"?
> -		  AC_LANG_DEFINES_PROVIDED])
> +AC_LANG_CONFTEST([AC_LANG_SOURCE([[Does cpp preserve   "whitespace"?]])])
>  if test `${RAWCPP} < conftest.$ac_ext | grep -c 'preserve   \"'` -eq 1 ; then
>  	AC_MSG_RESULT([no])
>  else

Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list