<!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 Sat, 2010-01-23 at 08:21 -0800, Dan Nicholson wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
On Sat, Jan 23, 2010 at 7:51 AM, Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt; wrote:
&gt; The man pages source may be supplied in the tarball or built
&gt; from git. The makefile needs to take that into consideration
&gt; and adjust the targets accordingly.
&gt;
&gt; Signed-off-by: Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt;

Thanks for doing this. One thing I'd like to point out is that we're
not allowing building of the man pages, but rather installing the
prebuilt pages.
</PRE>
</BLOCKQUOTE>
Not quite, there is a .man --&gt; .3 conversion, but not important enough to be considered a &quot;build&quot; process.
<BLOCKQUOTE TYPE=CITE>
<PRE>

&gt; ---
&gt; &nbsp;configure.ac &nbsp; &nbsp;| &nbsp; &nbsp;5 +++++
&gt; &nbsp;man/Makefile.am | &nbsp; &nbsp;2 +-
&gt; &nbsp;2 files changed, 6 insertions(+), 1 deletions(-)
&gt;
&gt; diff --git a/configure.ac b/configure.ac
&gt; index 2c898e8..1c95e89 100644
&gt; --- a/configure.ac
&gt; +++ b/configure.ac
&gt; @@ -45,6 +45,11 @@ XTST_CFLAGS=&quot;$CWARNFLAGS $XTST_CFLAGS&quot;
&gt; &nbsp;AC_SUBST(XTST_CFLAGS)
&gt; &nbsp;AC_SUBST(XTST_LIBS)
&gt;
&gt; +# Determine if the source for man pages is available
&gt; +# It may already be present (tarball) or can be generated using xmlto
&gt; +AM_CONDITIONAL([HAVE_MANPAGE_SRC], [test -f &quot;$srcdir/man/XTestQueryExtension.man&quot; \
&gt; +|| test &quot;x$have_xmlto&quot; = xyes])

Can you just wrap at the comma so the test isn't broken up? Also,
please indent on continued lines.

AM_CONDITIONAL([HAVE_MANPAGE_SRC],
    [test -f &quot;$srcdir/man/XTestQueryExtension.man&quot; || test
&quot;x$have_xmlto&quot; = xyes])

I'm not sure HAVE_MANPAGE_SRC is the best name. The source is the xml
and it's always there. Rather, this test determines whether we're
going to install the pages. I would call this INSTALL_MANPAGES or
something. Not a big deal, though.

&gt; +
&gt; &nbsp;dnl Allow checking code with lint, sparse, etc.
&gt; &nbsp;XORG_WITH_LINT
&gt; &nbsp;XORG_LINT_LIBRARY([Xtst])
&gt; diff --git a/man/Makefile.am b/man/Makefile.am
&gt; index 67bcd3f..789cd68 100644
&gt; --- a/man/Makefile.am
&gt; +++ b/man/Makefile.am
&gt; @@ -46,7 +46,7 @@ CLEANFILES = $(libman_DATA) $(BUILT_SOURCES)
&gt;
&gt; &nbsp;MAINTAINERCLEANFILES = $(libman_PRE)
&gt;
&gt; -if HAVE_XMLTO
&gt; +if HAVE_MANPAGE_SRC
&gt; &nbsp;libman_DATA = $(libman_PRE:man=@LIB_MAN_SUFFIX@)
&gt; &nbsp;endif

</PRE>
</BLOCKQUOTE>
<BR>
I agree with your comments, I ran out of creativity.
<BLOCKQUOTE TYPE=CITE>
<PRE>
--
Dan
</PRE>
</BLOCKQUOTE>
</BODY>
</HTML>