[prev in list] [next in list] [prev in thread] [next in thread] 

List:       autoconf
Subject:    Re: tcc 0.9.28rc testing: bug in autoconf 2.71 with AC_CHECK_DEFINE
From:       Detlef Riekenberg <wine.dev () web ! de>
Date:       2023-09-28 0:37:42
Message-ID: trinity-15740def-733e-43e1-bc95-173e1166bd66-1695861462220 () 3c-app-webde-bap07
[Download RAW message or body]

> > On 24/9/23 11:03, Nick Bowler wrote:
> >> The word AC_CHECK_DEFINE is not found anywhere in the Autoconf
> >> source code or documentation.
> >
> > My guess would be that the 3rd party is the autoconf archive because
> > they provide both AX_CHECK_DEFINE and AC_CHECK_DEFINE
> >
> > http://git.savannah.gnu.org/gitweb/?p=autoconf-archive.git;a=blob_plain;f=m4/ax_check_define.m4
>
> OK, I see.

Thanks Peter.
I can confirm, the "autoconf-archive" - Package is installed here.


> Ignoring the fact that the this macro definition flagrantly disregards
> the Autoconf reserved namespace...

That trap catched me.

> AC_CHECK_DEFINE here is quoting
> inconsistently: $1 is double-quoted in the argument to AC_LANG_PROGRAM,
> but it is only single-quoted in the arguments of AS_VAR_PUSHDEF and
> AC_CACHE_CHECK.

Oh, thanks for the in deepth analysis.
I will check to see, how and where i can report this bug.

> So no amount of quoting at the call site will ever solve the problem for Detlef.

The suggested workarounds should help.

> We can quote __unix__ correctly for AC_LANG_PROGRAM, or we can
> quote it for the other expansions, but never both at the same time.

>  For example:
>  AC_CHECK_DEFINE([__unix@&t@__], [...])

>   m4_ifdef([__unix__], [m4_define([__unix__], [[__unix__]])])dnl
>   AC_CHECK_DEFINE([__unix__], [...])

I will try your suggested workarounds.

> I don't think there is any regression in Autoconf here, I don't see any
> significant difference in behaviour between Autoconf 2.72c, 2.71 or
> 2.69.

I was not sure, where the bug belongs, and with the "AC_" prefix, i started here.

--
Regards ... Detlef


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic