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

[Dave Hart]

On Sat, Mar 19, 2011 at 04:27 UTC, Ralf Corsepius wrote:
> My use case is to set the version number consistently inside of a package
> (package suite) which consists of several sub-packages.
>
> To achieve this, I have two aclocal *.m4s containing something similar to
> this:
> aclocal/version.m4:
> AC_DEFUN([XXXX_VERSIONING],
> m4_define([_XXXX_VERSION],[1.2.99.0]))
> m4_define([_XXXX_API],[1.3])
>
> aclocal/init.m4:
> AC_DEFUN([XXXX_INIT],
> [AC_REQUIRE([XXXX_VERSIONING
> AC_PREFIX_DEFAULT([/opt/XXXX-][_XXXX_API])
> ])
>
> ... accompanied by this in configure.ac's:
> AC_INIT([XXXX],[_XXXX_VERSION],[http://www.XXXX.org/bugzilla])

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.

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],
    [http://bugs.ntp.org./],
    [],
    [http://www.ntp.org./]dnl
)

Cheers,
Dave Hart