[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:
> <>
> 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

Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]