<!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 Sun, 2010-01-17 at 21:42 +0100, Matthieu Herrb wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
On Tue, Jan 12, 2010 at 07:08:49PM -0500, Gaetan Nadon wrote:
&gt; From: Matthieu Herrb &lt;<A HREF="mailto:matthieu.herrb@laas.fr">matthieu.herrb@laas.fr</A>&gt;
&gt; 
&gt; This also allow to configure with --without-xmlto to ignore
&gt; a 3rd party xmlto tool on systems that normally don't have it,
&gt; in order to have reproducable builds.
&gt; 
&gt; Reviewed-by: R&#233;mi Cardona &lt;<A HREF="mailto:remi@gentoo.org">remi@gentoo.org</A>&gt;
&gt; Signed-off-by: Matthieu Herrb &lt;<A HREF="mailto:matthieu.herrb@laas.fr">matthieu.herrb@laas.fr</A>&gt;
&gt; Signed-off-by: Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt;

Reviewed-by: Matthieu Herrb &lt;<A HREF="mailto:matthieu.herrb@laas.fr">matthieu.herrb@laas.fr</A>&gt;

Gaetant or R&#233;mi Can you take care of committing it ? I'm a bit running 
out of time these days.
</PRE>
</BLOCKQUOTE>
Sure. I will.<BR>
<BR>
<BLOCKQUOTE TYPE=CITE>
<PRE>

&gt; ---
&gt;  xorg-macros.m4.in |   51 +++++++++++++++++++++++++++++++++++++++++++++++++++
&gt;  1 files changed, 51 insertions(+), 0 deletions(-)
&gt; 
&gt; diff --git a/xorg-macros.m4.in b/xorg-macros.m4.in
&gt; index db2a62d..b226262 100644
&gt; --- a/xorg-macros.m4.in
&gt; +++ b/xorg-macros.m4.in
&gt; @@ -310,6 +310,57 @@ AC_SUBST(MAKE_PDF)
&gt;  AC_SUBST(MAKE_HTML)
&gt;  ]) # XORG_CHECK_DOCBOOK
&gt;  
&gt; +# XORG_WITH_XMLTO
&gt; +# ----------------
&gt; +# Minimum version: 1.5.0
&gt; +#
&gt; +# Documentation tools are not always available on all platforms and sometimes
&gt; +# not at the appropriate level. This macro enables a module to test for the
&gt; +# presence of the tool and obtain it's path in separate variables. Coupled with
&gt; +# the --with-xmlto option, it allows maximum flexibilty in making decisions
&gt; +# as whether or not to use the xmlto package.
&gt; +#
&gt; +# Interface to module:
&gt; +# HAVE_XMLTO:         used in makefiles to conditionally generate documentation
&gt; +# XMLTO:        returns the path of the xmlto program found
&gt; +#                returns the path set by the user in the environment
&gt; +# --with-xmlto:        'yes' user instructs the module to use xmlto
&gt; +#                'no' user instructs the module not to use xmlto
&gt; +#
&gt; +# If the user sets the value of XMLTO, AC_PATH_PROG skips testing the path.
&gt; +#
&gt; +AC_DEFUN([XORG_WITH_XMLTO],[
&gt; +AC_ARG_VAR([XMLTO], [Path to xmlto command])
&gt; +AC_ARG_WITH(xmlto,
&gt; +        AS_HELP_STRING([--with-xmlto],
&gt; +           [Use xmlto to regenerate documentation (default: yes, if installed)]),
&gt; +           [use_xmlto=$withval], [use_xmlto=auto])
&gt; +
&gt; +if test &quot;x$use_xmlto&quot; = x&quot;auto&quot;; then
&gt; +   AC_PATH_PROG([XMLTO], [xmlto])
&gt; +   if test &quot;x$XMLTO&quot; = &quot;x&quot;; then
&gt; +        AC_MSG_WARN([xmlto not found - documentation targets will be skipped])
&gt; +        have_xmlto=no
&gt; +   else
&gt; +        have_xmlto=yes
&gt; +   fi
&gt; +elif test &quot;x$use_xmlto&quot; = x&quot;yes&quot; ; then
&gt; +   AC_PATH_PROG([XMLTO], [xmlto])
&gt; +   if test &quot;x$XMLTO&quot; = &quot;x&quot;; then
&gt; +        AC_MSG_WARN([--with-xmlto=yes specified but xmlto not found in PATH])
&gt; +   fi
&gt; +   have_xmlto=yes
&gt; +elif test &quot;x$use_xmlto&quot; = x&quot;no&quot; ; then
&gt; +   if test &quot;x$XMLTO&quot; != &quot;x&quot;; then
&gt; +      AC_MSG_WARN([ignoring XMLTO environment variable since --with-xmlto=no was specified])
&gt; +   fi
&gt; +   have_xmlto=no
&gt; +else
&gt; +   AC_MSG_ERROR([--with-xmlto expects 'yes' or 'no'])
&gt; +fi
&gt; +AM_CONDITIONAL([HAVE_XMLTO], [test &quot;$have_xmlto&quot; = yes])
&gt; +]) # XORG_CHECK_XMLTO
&gt; +
&gt;  # XORG_CHECK_MALLOC_ZERO
&gt;  # ----------------------
&gt;  # Minimum version: 1.0.0
&gt; -- 
&gt; 1.6.0.4
&gt; 
&gt; I took the liberty of submitting version 3 for review.
&gt; 
&gt;  New comment section
&gt;  Changed my mind about 'yes' + 'missing xmlto' = error. Not for this macro to decide. WARN instead.
&gt;  Changed user messages
&gt;  Last else simplified: option is not used for xmlto path, the XMLTO env var is. It's 'yes' or 'no'. 
&gt;  The behavior of 'yes' and 'auto' is now the same, but it may changed in  the future.
&gt; 
&gt; I intend to produce one of asciidoc and doxygen. 
&gt; 

</PRE>
</BLOCKQUOTE>
</BODY>
</HTML>