<!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 Fri, 2010-01-22 at 09:39 -0800, Dan Nicholson wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
On Fri, Jan 22, 2010 at 6:34 AM, Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt; wrote:
&gt; The default is passiveTex which does not work.
&gt; Add XMLFOP and HAVE_XMLFOP to skip pdf if fop is missing.
&gt;
&gt; Signed-off-by: Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt;
&gt; ---
&gt; &nbsp;Makefile.am &nbsp;| &nbsp; &nbsp;7 +++++--
&gt; &nbsp;configure.ac | &nbsp; &nbsp;9 +++++++++
&gt; &nbsp;2 files changed, 14 insertions(+), 2 deletions(-)
&gt;
&gt; diff --git a/Makefile.am b/Makefile.am
&gt; index fe4639c..41abd05 100644
&gt; --- a/Makefile.am
&gt; +++ b/Makefile.am
&gt; @@ -175,7 +175,10 @@ EXTRA_DIST += doc/xfs-design.xml doc/xfs-design.css doc/xfs-design.xsl
&gt;
&gt; &nbsp;if MAKE_DEVEL_DOCS
&gt; &nbsp;if HAVE_XMLTO
&gt; -noinst_DATA = doc/xfs-design/index.html doc/xfs-design.pdf doc/xfs-design.txt
&gt; +noinst_DATA = doc/xfs-design/index.html doc/xfs-design.txt
&gt; +if HAVE_XMLFOP
&gt; +noinst_DATA += doc/xfs-design.pdf
&gt; +endif HAVE_XMLFOP
&gt; &nbsp;CLEANFILES += $(noinst_DATA)
&gt;
&gt; &nbsp;doc/xfs-design/index.html: $(srcdir)/doc/xfs-design.xml $(srcdir)/doc/xfs-design.xsl doc/xfs-design/xfs-design.css
&gt; @@ -190,7 +193,7 @@ clean-local:
&gt; &nbsp; &nbsp; &nbsp; &nbsp;-rm -rf doc/xfs-design
&gt;
&gt; &nbsp;doc/xfs-design.pdf: $(srcdir)/doc/xfs-design.xml $(srcdir)/doc/xfs-design.xsl
&gt; - &nbsp; &nbsp; &nbsp; $(AM_V_GEN)$(XMLTO) -m $(srcdir)/doc/xfs-design.xsl -o doc pdf $(srcdir)/doc/xfs-design.xml
&gt; + &nbsp; &nbsp; &nbsp; $(AM_V_GEN)$(XMLTO) --with-fop -m $(srcdir)/doc/xfs-design.xsl -o doc pdf $(srcdir)/doc/xfs-design.xml
&gt;
&gt; &nbsp;doc/xfs-design.txt: $(srcdir)/doc/xfs-design.xml $(srcdir)/doc/xfs-design.xsl
&gt; &nbsp; &nbsp; &nbsp; &nbsp;$(AM_V_GEN)$(XMLTO) -m $(srcdir)/doc/xfs-design.xsl -o doc txt $(srcdir)/doc/xfs-design.xml
&gt; diff --git a/configure.ac b/configure.ac
&gt; index b429ed5..85bcac6 100644
&gt; --- a/configure.ac
&gt; +++ b/configure.ac
&gt; @@ -95,6 +95,15 @@ AC_ARG_WITH(default-font-path, AS_HELP_STRING([--with-default-font-path=PATH], [
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; [ FONTPATH=&quot;${DEFAULT_FONT_PATH}&quot; ])
&gt; &nbsp;AC_SUBST([FONTPATH])
&gt;
&gt; +AC_PATH_PROG([XMLFOP], [fop])
&gt; +if test &quot;x$XMLFOP&quot; = &quot;x&quot;; then
&gt; + &nbsp; AC_MSG_WARN([fop not found - pdf documentation targets will be skipped])
&gt; + &nbsp; have_xmlfop=no

Actually, AC_PATH_PROG will make the variable XMLFOP=&quot;no&quot; when fop is
not found, not an empty string. So, the test should be &quot;x$XMLFOP&quot; =
xno.
</PRE>
</BLOCKQUOTE>
<BR>
It does not seem to behave that way on my system:<BR>
AC_PATH_PROG([XMLFOP], [fop])<BR>
AS_ECHO(&quot;----&gt;&quot;$XMLFOP&quot;&lt;-----&quot;)<BR>
yields:
<BLOCKQUOTE>
<PRE>
checking for fop... no
----&gt;&lt;-----
</PRE>
</BLOCKQUOTE>
Autoconf 2.63 on Debian<BR>
<BR>
In config.log:<BR>
HAVE_XMLFOP_FALSE=''&nbsp;&nbsp;&nbsp; <BR>
HAVE_XMLFOP_TRUE='#'&nbsp;&nbsp;&nbsp; Means makefile code is commented out if true<BR>
<BR>
In Makefile:<BR>
#am__append_1 = doc/xfs-design.pdf&nbsp;&nbsp;&nbsp; This target is commented out, no pdf gets generated<BR>
<BR>
Thanks
<BLOCKQUOTE TYPE=CITE>
<PRE>

&gt; +else
&gt; + &nbsp; have_xmlfop=yes
&gt; +fi
&gt; +AM_CONDITIONAL([HAVE_XMLFOP], [test &quot;$have_xmlfop&quot; = yes])

Other than that, it looks fine to me.

Reviewed-by: Dan Nicholson &lt;<A HREF="mailto:dbn.lists@gmail.com">dbn.lists@gmail.com</A>&gt;
</PRE>
</BLOCKQUOTE>
</BODY>
</HTML>