Stefano Lattarini | 19 May 2012 10:25
Picon

[PATCH] general: deprecate 'configure.in' as autoconf input

It has been years since that has been deprecated in the documentation,
in favour of 'configure.ac':

  Previous versions of Autoconf promoted the name configure.in, which
  is somewhat ambiguous (the tool needed to process this file is not
  described by its extension), and introduces a slight confusion with
  config.h.in and so on (for which '.in' means "to be processed by
  configure"). Using configure.ac is now preferred.

It's now time to start giving runtime warning about the use of
'configure.in', so that support for it can be removed in future
versions of autoconf/automake.

* lib/Autom4te/Configure_ac.pm: Issue a warning in the 'obsolete'
category if 'configure.in' is detected.  Since this module is synced
from Automake, this change is to be backported there (and will be
soon).
* doc/autoconf.texi: Update.
* tests/tools.at: Adjust to avoid spurious failures.

Signed-off-by: Stefano Lattarini <stefano.lattarini <at> gmail.com>
---

 OK to apply?  I plan to backport the change done to Configure_ac.pm
 to the Automake's master copy as soon as this patch is accepted into
 the autoconf repository.

 Regards,
   Stefano

(Continue reading)

Stefano Lattarini | 22 May 2012 12:29
Picon

Re: [PATCH] general: deprecate 'configure.in' as autoconf input

On 05/19/2012 10:25 AM, Stefano Lattarini wrote:
> It has been years since that has been deprecated in the documentation,
> in favour of 'configure.ac':
> 
>   Previous versions of Autoconf promoted the name configure.in, which
>   is somewhat ambiguous (the tool needed to process this file is not
>   described by its extension), and introduces a slight confusion with
>   config.h.in and so on (for which '.in' means "to be processed by
>   configure"). Using configure.ac is now preferred.
> 
> It's now time to start giving runtime warning about the use of
> 'configure.in', so that support for it can be removed in future
> versions of autoconf/automake.
> 
> * lib/Autom4te/Configure_ac.pm: Issue a warning in the 'obsolete'
> category if 'configure.in' is detected.  Since this module is synced
> from Automake, this change is to be backported there (and will be
> soon).
> * doc/autoconf.texi: Update.
> * tests/tools.at: Adjust to avoid spurious failures.
> 
> Signed-off-by: Stefano Lattarini <stefano.lattarini <at> gmail.com>
> ---
> 
>  OK to apply?  I plan to backport the change done to Configure_ac.pm
>  to the Automake's master copy as soon as this patch is accepted into
>  the autoconf repository.
> 
>  Regards,
>    Stefano
(Continue reading)

Paul Eggert | 22 May 2012 18:24
Favicon

Re: [PATCH] general: deprecate 'configure.in' as autoconf input

On 05/22/2012 03:29 AM, Stefano Lattarini wrote:
> +** The use of the long-deprecated name 'configure.in' for the autoconf
>> +   input file now elicits a warning in the 'obsolete' category.
>> 

This change is fine with me.  The only package I know that's
still using configure.in is Emacs, and I can rename that.

Eric Blake | 22 May 2012 18:29
Picon
Favicon
Gravatar

Re: [PATCH] general: deprecate 'configure.in' as autoconf input

On 05/22/2012 10:24 AM, Paul Eggert wrote:
> On 05/22/2012 03:29 AM, Stefano Lattarini wrote:
>> +** The use of the long-deprecated name 'configure.in' for the autoconf
>>> +   input file now elicits a warning in the 'obsolete' category.
>>>
> 
> This change is fine with me.  The only package I know that's
> still using configure.in is Emacs, and I can rename that.

Agreed.  Let's go for it.  We still need to figure out if it is worth
sharing the .pm files in gnulib as a common upstream to both autoconf
and automake, but I haven't had time to look into that yet.

--

-- 
Eric Blake   eblake <at> redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Stefano Lattarini | 23 May 2012 00:13
Picon

Re: [PATCH] general: deprecate 'configure.in' as autoconf input

On 05/22/2012 06:29 PM, Eric Blake wrote:
> On 05/22/2012 10:24 AM, Paul Eggert wrote:
>> On 05/22/2012 03:29 AM, Stefano Lattarini wrote:
>>> +** The use of the long-deprecated name 'configure.in' for the autoconf
>>>> +   input file now elicits a warning in the 'obsolete' category.
>>>>
>>
>> This change is fine with me.  The only package I know that's
>> still using configure.in is Emacs, and I can rename that.
> 
> Agreed.  Let's go for it.
> 
Thanks Eric nad Paul for your feedback.  I've pushed the patch to master.

Regards,
  Stefano


Gmane