> Bruce, how about this?  I'd actually rather see the whole
   > version.m4 mess be removed, but this is a decent fix for now.

   A good and proper define-in-one-place fix for version stamps is a
   really good idea.  Sticking the original version string into an
   AC_INIT() m4 macro is _not_ a good idea.  It needs to be in a
   readily accessible place.

Which is exactly what AC_INIT does, it sets PACKAGE_VERSION,
PACKAGE_NAME and other such variables that should be used instead.

   > ! AC_INIT([GNU 
sharutils],m4_defn([sharutils_version]),m4_defn([sharutils_eaddr]))

   I was about to ask of sharutils_eaddr needed an "m4_defn()"
   wrapper, too.

:-)

Happy hacking!


Reply via email to