automake-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] {maint} distcheck: add support for AM_DISTCHECK_CONFIGURE_FL


From: Eric Blake
Subject: Re: [PATCH] {maint} distcheck: add support for AM_DISTCHECK_CONFIGURE_FLAGS
Date: Wed, 15 Jun 2011 11:04:56 -0600
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110428 Fedora/3.1.10-1.fc14 Lightning/1.0b3pre Mnenhy/0.8.3 Thunderbird/3.1.10

On 06/15/2011 02:18 AM, Ralf Corsepius wrote:
> On 06/15/2011 09:40 AM, Stefano Lattarini wrote:
>> On Tuesday 14 June 2011, Ralf Wildenhues wrote:
>>> * Stefano Lattarini wrote on Fri, Jun 10, 2011 at 05:33:39PM CEST:
> 
>>> I would suggest to at least discourage using this in the documentation.
>>>
>> ... I agree, and I will make the change soon.  Maybe I can find a
>> couple of
>> examples of where the AM_DISTCHECK_CONFIGURE_FLAGS could be useful *and*
>> legitimate; which would make my change more meaningful.
> 
> DISTCHECK_CONFIGURE_FLAGS is useful in situations when a plain
> "./configure" is not meaningful to a source tree, i.e. when a
> source-tree mandatorily requires some configuration argument.

Such a source-tree is violating GNU Coding Standards.  On the other
hand, automake strives to be useful to more than just GCS-compliant
packages.

As for a valid use of AM_DISTCHECK_CONFIGURE_FLAGS, the m4 package
normally configures --without-changeword, but it is useful to have 'make
distcheck' exercise the --with-changeword option to ensure that the code
still compiles for those few people that want to enable the experimental
configure option.  m4 also uses it to stress-test --program-prefix=g
(since at one point m4 had a problem where 'make installcheck' was
assuming it could blindly test "m4" rather than the just-installed "gm4"
when that renaming option was in effect - see m4.git commit fb04a26fa).
 I've previously been using DISTCHECK_CONFIGURE_FLAGS for that purpose,
but like the idea of AM_DISTCHECK_CONFIGURE_FLAGS better.

-- 
Eric Blake   address@hidden    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

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