[Top][All Lists]

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

Re: [PATCH] AC_INIT: properly quote package name containing m4 macro

From: Eric Blake
Subject: Re: [PATCH] AC_INIT: properly quote package name containing m4 macro
Date: Thu, 17 Mar 2011 11:29:00 -0600
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv: Gecko/20101209 Fedora/3.1.7-0.35.b3pre.fc14 Lightning/1.0b3pre Mnenhy/0.8.3 Thunderbird/3.1.7

On 03/17/2011 11:19 AM, Stefano Lattarini wrote:
> So, even if I personally prefer the new behaviour your patch implements,
> the Automake testsuite is *not* a real use case for it.

Indeed - and an existing counterexample project that depends on
expansion will do a lot more than an enhancement request from automake
that automake won't be using for some time to come.

> On the other hand, what if a new project wants to name itself 'GNU index'
> or 'GNU eval'?  The current autoconf implementation would prevent it to
> do so, unless it uses an extra level of quoting:
>   AC_INIT([[GNU eval]], [1.0])
> But then, would autoconf really handle these extra quotes gracefully
> in all its codebase?

If we argue that the _correct_ fix is to expand the argument, then the
double quoting should just work uniformly everywhere (the fact that it
doesn't right now is part of the problem I'm referring to - autoconf is
inconsistent on whether the result should be expanded or literal).

Eric Blake   address@hidden    +1-801-349-2682
Libvirt virtualization library

Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to

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