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

[Eric Blake]

On 03/20/2011 11:32 AM, Dave Hart wrote:
> By my reading, Eric is proposing to require the first argument, the
> package name, be unexpanded.  You care about the 2nd argument being
> expanded, so I'm wondering if Eric's proposal impacts your use at all.

Not quite - my proposal was an all or none proposition, initially
leaning towards none.

> 
> If expanding AC_INIT's second argument would be impacted, I think NTP
> may be affected as well:
> 
> m4_include([sntp/m4/version.m4])
> AC_PREREQ([2.61])
> AC_INIT(
>     [ntp],
>     [VERSION_NUMBER],

But given that uses like this exist in the wild, I've changed my mind -
you are correct that the only reasonable approach is to expand all
arguments.  I will have to adjust the patch accordingly; the goal is that:

AC_INIT([[my-changequote-package]], [1.0])

should allow you to name a package with 'changequote' in the title, but
without extra [].

However, it will probably be a few days before I have a working approach.

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

signature.asc
Description: OpenPGP digital signature