Adjust version comparison to account for git snapshot numbers.

* lib/m4sugar/m4sugar.m4 (_m4_version_unletter): Also treat - as a
component separator.
* doc/autoconf.texi (Number processing Macros)
<m4_version_compare>: Document this change.
* tests/m4sugar.at (m4@&t@_version_compare): Test it.

Signed-off-by: Eric Blake <ebb9@byu.net>

diff --git a/ChangeLog b/ChangeLog
index e48da1fd73451653b8fef6475a7832ebbcc0b8b8..8124d3dc73e1f9c2dbae367f73a0fcefa901f514 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2007-11-03  Eric Blake  <ebb9@byu.net>
+
+       Adjust version comparison to account for git snapshot numbers.
+       * lib/m4sugar/m4sugar.m4 (_m4_version_unletter): Also treat - as a
+       component separator.
+       * doc/autoconf.texi (Number processing Macros)
+       <m4_version_compare>: Document this change.
+       * tests/m4sugar.at (m4@&t@_version_compare): Test it.
+
 2007-10-30  Bruno Haible  <bruno@clisp.org>
 
        * lib/autoconf/types.m4 (_AC_TYPE_LONG_LONG_SNIPPET): New macro,

diff --git a/doc/autoconf.texi b/doc/autoconf.texi
index f05b2ec485e70eaf3da898c7ba04efd9b115b363..73077a03206a2769021b7ca3febc6a770872f85e 100644 (file)
--- a/doc/autoconf.texi
+++ b/doc/autoconf.texi
@@ -11182,10 +11182,19 @@ usability limitations that were not lifted until Autoconf 2.62.  Compare
 the version strings @var{version-1} and @var{version-2}, and expand to
 @samp{-1} if @var{version-1} is smaller, @samp{0} if they are the same,
 or @samp{1} @var{version-2} is smaller.  Version strings must be a list
-of elements separated by @samp{.}, where each element is a number along
-with optional case-insensitive letters designating beta releases.  The
-comparison stops at the leftmost element that contains a difference,
-although a 0 element compares equal to a missing element.
+of elements separated by @samp{.}, @samp{,} or @samp{-}, where each
+element is a number along with optional case-insensitive letters
+designating beta releases.  The comparison stops at the leftmost element
+that contains a difference, although a 0 element compares equal to a
+missing element.
+
+It is permissible to include commit identifiers in @var{version}, such
+as an abbreviated SHA1 of the commit, provided there is still a
+monotonically increasing prefix to allow for accurate version-based
+comparisons.  For example, this paragraph was written when the
+development snapshot of autoconf claimed to be at version
+@samp{2.61a-248-dc51}, or 248 commits after the 2.61a release, with an
+abbreviated commit identification of @samp{dc51}.
 
 @example
 m4_version_compare([1.1], [2.0])
@@ -11202,6 +11211,10 @@ m4_version_compare([1.1pre], [1.1PRE])
 @result{}0
 m4_version_compare([1.1a], [1,10])
 @result{}-1
+m4_version_compare([2.61a], [2.61a-248-dc51])
+@result{}-1
+m4_version_compare([2.61b], [2.61a-248-dc51])
+@result{}1
 @end example
 @end defmac
 

diff --git a/lib/m4sugar/m4sugar.m4 b/lib/m4sugar/m4sugar.m4
index e4bf9b291cf9055290697daf37376dc2fefd53a6..e048dc2c66816ae847c0447f9eb1accd14e9c51f 100644 (file)
--- a/lib/m4sugar/m4sugar.m4
+++ b/lib/m4sugar/m4sugar.m4
@@ -2129,18 +2129,20 @@ m4_define([m4_sign],
 #   [2.61aa.b] -> [2.61+1.-1.[0r36:aa],+1.-1.[0r36:b]] -> 2.62.-1.370.1.-1.11
 #
 # This macro expects reasonable version numbers, but can handle double
-# letters and does not expand one-letter macros.  Inline constant expansions,
-# to avoid m4_defn overhead.  _m4_version_unletter is the real workhorse
-# used by m4_version_compare, but since [0r36:a] is less readable than 10,
-# we provide a wrapper for human use.
+# letters and does not expand any macros.  Original version strings can
+# use both `.' and `-' separators.
+#
+# Inline constant expansions, to avoid m4_defn overhead.
+# _m4_version_unletter is the real workhorse used by m4_version_compare,
+# but since [0r36:a] is less readable than 10, we provide a wrapper for
+# human use.
 m4_define([m4_version_unletter],
 [m4_map_sep([m4_eval], [.],
            m4_dquote(m4_dquote_elt(m4_unquote(_$0([$1])))))])
 m4_define([_m4_version_unletter],
-[m4_translit(m4_bpatsubst([[[$1]]], ]dnl
+[m4_bpatsubst(m4_translit([[[$1]]], [.-], [,,]),]dnl
 m4_dquote(m4_dquote(m4_defn([m4_cr_Letters])))[[+],
-                         [+1.-1.[0r36:\&]]),
-            [.], [,])])
+             [+1,-1,[0r36:\&]])])
 
 
 # m4_version_compare(VERSION-1, VERSION-2)

diff --git a/tests/m4sugar.at b/tests/m4sugar.at
index 9b1dc0eeb01f665bd727d71a21685302ec3cb4f0..163dea1f40c5f0e0ca8ef0d760cb5d3a47edd573 100644 (file)
--- a/tests/m4sugar.at
+++ b/tests/m4sugar.at
@@ -368,10 +368,12 @@ m4_version_compare([1.0], [1])
 m4_version_compare([1.0a], [1.0a])
 m4_version_compare([1.1a], [1.1a.1])
 m4_version_compare([1.10], [1.1a])
-m4_version_compare([1.1a], [1.1A])
+m4_version_compare([1-1a], [1,1A])
 m4_define([a], [oops])dnl
 m4_version_compare([1.1a], [1.1A])
 m4_version_compare([1z], [1aa])
+m4_version_compare([2.61a], [2.61a-248-dc51])
+m4_version_compare([2.61b], [2.61a-248-dc51])
 ]],
 [[-1
 1
@@ -385,6 +387,8 @@ m4_version_compare([1z], [1aa])
 0
 0
 -1
+-1
+1
 ]])
 
 AT_CLEANUP