[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] warn: new special syntax warning, hack for aclocal
From: |
Eric Blake |
Subject: |
Re: [PATCH] warn: new special syntax warning, hack for aclocal |
Date: |
Fri, 02 Nov 2012 14:22:24 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121016 Thunderbird/16.0.1 |
On 11/02/2012 01:49 PM, Stefano Lattarini wrote:
> Reference:
> <http://lists.gnu.org/archive/html/autoconf-patches/2012-11/msg00004.html>
>
> On 11/02/2012 07:34 PM, Stefano Lattarini wrote:
>> Well, it turns out that such warnings are coming from the autom4te
>> invocation, so I see no simple, non-hacky way to avoid them, unless
>> we add a knob to autom4te itself.
>>
>> So, autoconfers: would be OK with you to add a new warning "category"
>> to autom4te that would suppress *only* this specific warning?
>> The warning in question being
>>
>> AC_FOO is m4_require'd but not m4_defun'd
>>
>> that is emitted by the '_m4_require_call' internal macro (defined in
>> 'lib/m4sugar/m4sugar.m4').
>>
> Oh well, I went on and wrote a patch anyway. It should be good to go,
> maybe modulo the name of the new "singleton" warnings category; if anyone
> has a better name to suggest, shoot!
Hmm, your idea turned out to be relatively simple; fewer lines than the
attempt I had of adding a new file to some but not all of the autom4te
languages to conditionally control the warning.
> +++ b/lib/m4sugar/m4sugar.m4
> @@ -2080,13 +2080,18 @@ m4_if([$0], [m4_require], [[m4_defun]],
> [[AC_DEFUN]])['d macro])])]dnl
> #
> # This is called frequently, so minimize the number of macro invocations
> # by avoiding dnl and other overhead on the common path.
> +# The use of convoluted warning category 'm4require-without-m4defun' is
> +# an hack required by aclocal to support AC_CONFIG_MACRO_DIRS without
s/an hack/a hack/
(The rule of thumb is that if the 'h' is aspirated, as in hack, then you
use 'a'; if the 'h' is silent, as in hour, then use 'an')
ACK with that fix; and assuming I ever get around to reviewing the other
two patches that add AC_CONFIG_MACRO_DIRS in the first place (looking at
them now...)
--
Eric Blake address@hidden +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
Re: [PATCH 0/2] AC_CONFIG_MACRO_DIRS: implementation and documentation, Nick Bowler, 2012/11/02
- Re: [PATCH 0/2] AC_CONFIG_MACRO_DIRS: implementation and documentation, Stefano Lattarini, 2012/11/02
- Re: [PATCH 0/2] AC_CONFIG_MACRO_DIRS: implementation and documentation, Stefano Lattarini, 2012/11/02
- [PATCH] warn: new special syntax warning, hack for aclocal (was: Re: [PATCH 0/2] AC_CONFIG_MACRO_DIRS: implementation and documentation), Stefano Lattarini, 2012/11/02
- Re: [PATCH] warn: new special syntax warning, hack for aclocal,
Eric Blake <=
- Re: [PATCH] warn: new special syntax warning, hack for aclocal, Stefano Lattarini, 2012/11/02
Re: [PATCH 0/2] AC_CONFIG_MACRO_DIRS: implementation and documentation, Eric Blake, 2012/11/02