Bug in g-wrap.m4/acinclude.m4

Derek Atkins warlord@MIT.EDU
Thu, 9 Nov 2000 15:34:10 -0500


I found a bug in the g-wrap.m4 code (duplicated in acinclude.m4 in the
gnucash sources).  The problem is that it fails to properly compute
the minor and micro/patchlevel release sub-numbers of the required
version (always using the major number).

Here is a patch against acinclude.m4.  Rob, feel free to also apply
this patch against the g-wrap sources (editing the patch as
appropriate :)

Enjoy!

-derek

Index: acinclude.m4
===================================================================
RCS file: /home/cvs/cvsroot/gnucash/acinclude.m4,v
retrieving revision 1.2
diff -u -r1.2 acinclude.m4
--- acinclude.m4	2000/11/02 23:26:34	1.2
+++ acinclude.m4	2000/11/09 20:30:08
@@ -82,9 +82,9 @@
 major_required=`echo ${min_gwrap_version} |\
         sed 's/\([[0-9]]*\).\([[0-9]]*\).\([[0-9]]*\)/\1/'`
 minor_required=`echo ${min_gwrap_version} |\
-	sed 's/\([[0-9]]*\).\([[0-9]]*\).\([[0-9]]*\)/\1/'`
+	sed 's/\([[0-9]]*\).\([[0-9]]*\).\([[0-9]]*\)/\2/'`
 micro_required=`echo ${min_gwrap_version} |\
-	sed 's/\([[0-9]]*\).\([[0-9]]*\).\([[0-9]]*\)/\1/'`
+	sed 's/\([[0-9]]*\).\([[0-9]]*\).\([[0-9]]*\)/\3/'`
 
 if ${GUILE} -c "(cond ((> ${gwrap_major_version} ${major_required}) (exit 0))\
 	           ((< ${gwrap_major_version} ${major_required}) (exit 1))\