On Sep 12, 2007, at 8:23 PM, Eric Blake wrote:
Benoit SIGOURE <tsuna <at> lrde.epita.fr> writes:It looks great. The name m4_PACKAGE_VERSION is somewhat misleading however (as I understand it, it contains the current version of autoconf right? So why not name it like m4_AUTOCONF_VERSION or whatever?).Good point. There are other m4_PACKAGE_* macros that I did not document; basically, they all perform a hard-wired mapping to the contents of the PACKAGE_* macros from autoconf's configure script. I like your idea of making the public macro have a more obvious name. I'll rework my patch accordingly(as in, add m4_AUTOCONF_VERSION as the new documented alias, and leave m4_PACKAGE_VERSION available, but undocumented).
Sounds good. Cheers, -- Benoit Sigoure aka Tsuna EPITA Research and Development Laboratory
PGP.sig
Description: This is a digitally signed message part
