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).

-- 
Eric Blake





Reply via email to