<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/3.26.0">
</HEAD>
<BODY>
On Thu, 2010-03-04 at 12:29 -0800, Dan Nicholson wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
On Thu, Mar 4, 2010 at 11:09 AM, Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt; wrote:
&gt; On Thu, 2010-03-04 at 09:25 -0800, Dan Nicholson wrote:
&gt;
&gt; On Thu, Mar 4, 2010 at 9:08 AM, Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt; wrote:
&gt;&gt; On Wed, 2010-03-03 at 20:16 -0800, Dan Nicholson wrote:
&gt;&gt;
&gt;&gt; Adds an optional parameter to XORG_CHECK_XMLTO to enforce a minimum
&gt;&gt; version needed like the asciidoc check.
&gt;&gt;
&gt;&gt; Signed-off-by: Dan Nicholson &lt;<A HREF="mailto:dbn.lists@gmail.com">dbn.lists@gmail.com</A>&gt;
&gt;&gt; ---
&gt;&gt;  xorg-macros.m4.in |   16 +++++++++++++++-
&gt;&gt;  1 files changed, 15 insertions(+), 1 deletions(-)
&gt;&gt;
&gt;&gt; diff --git a/xorg-macros.m4.in b/xorg-macros.m4.in
&gt;&gt; index e9eecf2..016f169 100644
&gt;&gt; --- a/xorg-macros.m4.in
&gt;&gt; +++ b/xorg-macros.m4.in
&gt;&gt; @@ -304,7 +304,7 @@ AC_SUBST(MAKE_PDF)
&gt;&gt;  AC_SUBST(MAKE_HTML)
&gt;&gt;  ]) # XORG_CHECK_DOCBOOK
&gt;&gt;
&gt;&gt; -# XORG_WITH_XMLTO
&gt;&gt; +# XORG_WITH_XMLTO([MIN-VERSION])
&gt;&gt;  # ----------------
&gt;&gt;  # Minimum version: 1.5.0
&gt;&gt;  #
&gt;&gt; @@ -352,6 +352,20 @@ elif test &quot;x$use_xmlto&quot; = x&quot;no&quot; ; then
&gt;&gt;  else
&gt;&gt;     AC_MSG_ERROR([--with-xmlto expects 'yes' or 'no'])
&gt;&gt;  fi
&gt;&gt; +m4_ifval([$1],
&gt;&gt; +[if test &quot;$have_xmlto&quot; = yes; then
&gt;&gt; +    # scrape the xmlto version
&gt;&gt; +    AC_MSG_CHECKING([the xmlto version])
&gt;&gt; +    xmlto_version=`$XMLTO --version 2&gt;/dev/null | cut -d' ' -f3`
&gt;&gt; +    AC_MSG_RESULT([$xmlto_version])
&gt;&gt; +    AS_VERSION_COMPARE([$xmlto_version], [$1],
&gt;&gt; +        [if test &quot;x$use_xmlto&quot; = xauto; then
&gt;&gt; +            AC_MSG_WARN([xmlto version $xmlto_version found, but $1
&gt;&gt; needed])
&gt;&gt; +            have_xmlto=no
&gt;&gt; +        else
&gt;&gt; +            AC_MSG_ERROR([xmlto version $xmlto_version found, but $1
&gt;&gt; needed])
&gt;&gt; +        fi])
&gt;&gt; +fi])
&gt;&gt;  AM_CONDITIONAL([HAVE_XMLTO], [test &quot;$have_xmlto&quot; = yes])
&gt;&gt;  ]) # XORG_CHECK_XMLTO
&gt;&gt;
&gt;&gt;
&gt;&gt; Tested-by: Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt;
&gt;&gt;
&gt;&gt; If you were considering bumping the version number of the util-macros
&gt;&gt; package and releasing it, I'll soon post a patch
&gt;&gt; that will require a version bump as well.
&gt;
&gt; Well, I guess we have to bump it to use it in libXi. I don't know if
&gt; that's a minor or patch level bump. I guess minor since we're changing
&gt; the interface to the macros.
&gt;
&gt; That would be 1.7.0 then. The feature will work anyway if libXi is
&gt; configured against a version of util-macros containing the feature. The
&gt; benefit of bumping the number in both packages is to &quot;enforce&quot; that the
&gt; feature is used.

I think I'd rather just bump the patch level since there's not a new macro here.

</PRE>
</BLOCKQUOTE>
Ah, ok. I thought you wanted to bump XORG_MACROS_VERSION(1.5) to force libXi to configure against the latest util-macros with your changes because you wrote &quot;we have to bump it to use it in libXi&quot;. <BR>
<BR>
I think bumping the patch level is good enough in this situation. If someone configures latest libXi against util-macros @ 1.5, for example, the parameter is ignored.<BR>
<BR>
<BR>
<BR>
<BR>
<BLOCKQUOTE TYPE=CITE>
<PRE>
--
Dan
</PRE>
</BLOCKQUOTE>
</BODY>
</HTML>