[PATCH xts] doc: fix man page installation and distcheck
Peter Hutterer
peter.hutterer at who-t.net
Thu Jul 26 22:02:13 PDT 2012
On Thu, Jul 26, 2012 at 09:37:54PM -0700, Aaron Plattner wrote:
> On 07/26/2012 09:23 PM, Peter Hutterer wrote:
> >On Wed, Jul 25, 2012 at 11:29:02PM -0700, Aaron Plattner wrote:
> >>When asciidoc and xmlto are available, HAVE_DOCTOOLS. This turns on code that
> >>defines miscmandir = $(MISC_MAN_DIR) and sets miscman_DATA =
> >>XTS.$(MISC_MAN_SUFFIX). However, $(MISC_MAN_DIR) is never defined, presumably
> >>because the corresponding lines from xorg-macros.m4.
> >
> >simple fix appears to be to use XORG_DEFAULT_OPTIONS (or, alternatively,
> >XORG_MANPAGE_SECTIONS)
>
> I thought XTS wasn't supposed to use xorg macros, to keep it
> independent of X11 implementations. For build stuff like this it
> doesn't seem like it would hurt, though. It's not like util/macros
> is a huge burdensome dependency.
anyone building X already needs the macros (and I suspect that a host that
builds XTS is likely to have X built as well). Plus, the macros are pretty
much standalone and don't interfere with anything else. So I think it's safe
enough to require them here. The maintainer efforts should certainly go
down, though we need to keep an eye out for any new -Werror flags.
Cheers,
Peter
>
> If no one objects, I'll go with your version.
>
> -- Aaron
>
> >>In addition, the target file XTS.7 and intermediate file XTS.txt are not cleaned
> >>up during distclean because XTS.7 is not listed in any clean target. This
> >>causes distcheck failures. Fix this by moving all of the man page intermediate
> >>files to CLEANFILES.
> >>
> >>Finally, use $^ and $@ rather than $(srcdir)/README and XTS.txt in the rule to
> >>copy the README to XTS.txt.
> >>
> >>Signed-off-by: Aaron Plattner <aplattner at nvidia.com>
> >>---
> >> Makefile.am | 4 ++--
> >> configure.ac | 4 ++++
> >> 2 files changed, 6 insertions(+), 2 deletions(-)
> >>
> >>diff --git a/Makefile.am b/Makefile.am
> >>index 87040d2..9083bb6 100644
> >>--- a/Makefile.am
> >>+++ b/Makefile.am
> >>@@ -32,7 +32,7 @@ miscman_DATA = $(xtsman_pre:.man=.$(MISC_MAN_SUFFIX))
> >> SUFFIXES = .man .$(MISC_MAN_SUFFIX) .txt .xml
> >>
> >> XTS.txt: README
> >>- cp $(srcdir)/README XTS.txt
> >>+ cp $^ $@
> >
> >Acked-by: Peter Hutterer <peter.hutterer at who-t.net> for this hunk
> >
> >>
> >> .man.$(MISC_MAN_SUFFIX):
> >> sed -e 's|miscmansuffix|$(MISC_MAN_SUFFIX)|' < $< > $@
> >>@@ -42,6 +42,6 @@ XTS.txt: README
> >> $(XMLTO) man $<
> >> mv -f $(@:.man=.miscmansuffix) $@
> >>
> >>-MAINTAINERCLEANFILES = *.xml $(xtsman_pre) $(xtsman_txt)
> >>+CLEANFILES = $(miscman_DATA) $(xtsman_xml) $(xtsman_pre) $(xtsman_txt)
> >
> >Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net> for this hunk :)
> >
> >> endif
> >>
> >>diff --git a/configure.ac b/configure.ac
> >>index 07abff3..0601758 100644
> >>--- a/configure.ac
> >>+++ b/configure.ac
> >>@@ -141,6 +141,10 @@ else
> >> esac
> >> fi
> >> AC_SUBST(MISC_MAN_SUFFIX)
> >>+ if test x$MISC_MAN_DIR = x ; then
> >>+ MISC_MAN_DIR='$(mandir)/man$(MISC_MAN_SUFFIX)'
> >>+ fi
> >>+ AC_SUBST(MISC_MAN_DIR)
> >
> >how about this patch here, it passes make distcheck. too lazy to look up
> >with xorg-macros version we really need here, so I just used the latest
> >
> > From 878d66806b2b0383813467e7e3860c8361484ad0 Mon Sep 17 00:00:00 2001
> >From: Peter Hutterer <peter.hutterer at who-t.net>
> >Date: Fri, 27 Jul 2012 14:20:25 +1000
> >Subject: [PATCH] Use xorg-macros to drag in man page build dependencies
> >
> >And make sure to remove generated man page files during make clean.
> >
> >Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> >---
> > Makefile.am | 2 +-
> > configure.ac | 27 +++++++++------------------
> > 2 files changed, 10 insertions(+), 19 deletions(-)
> >
> >diff --git a/Makefile.am b/Makefile.am
> >index 87040d2..59f9951 100644
> >--- a/Makefile.am
> >+++ b/Makefile.am
> >@@ -42,6 +42,6 @@ XTS.txt: README
> > $(XMLTO) man $<
> > mv -f $(@:.man=.miscmansuffix) $@
> >
> >-MAINTAINERCLEANFILES = *.xml $(xtsman_pre) $(xtsman_txt)
> >+CLEANFILES = *.xml $(xtsman_pre) $(xtsman_txt) $(miscman_DATA)
> > endif
> >
> >diff --git a/configure.ac b/configure.ac
> >index 07abff3..777d652 100644
> >--- a/configure.ac
> >+++ b/configure.ac
> >@@ -30,6 +30,12 @@ AC_CONFIG_SRCDIR([Makefile.am])
> > AC_CONFIG_AUX_DIR(.)
> > AM_INIT_AUTOMAKE([dist-bzip2 foreign])
> >
> >+# Initialize X.Org macros
> >+m4_ifndef([XORG_MACROS_VERSION],
> >+ [m4_fatal([must install xorg-macros 1.17 or later before running autoconf/autogen])])
> >+XORG_MACROS_VERSION(1.17)
> >+XORG_DEFAULT_OPTIONS
> >+
> > AM_MAINTAINER_MODE
> >
> > AM_CONFIG_HEADER([config.h])
> >@@ -121,26 +127,11 @@ if test "x$PERL" = x || test "x$XSET" = x || test "x$XDPYINFO" = x; then
> > AC_MSG_ERROR([perl, xset and xdpyinfo are required to run xts-config])
> > fi
> >
> >-m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES([yes])],
> >- [AC_SUBST([AM_DEFAULT_VERBOSITY], [1])])
> >-
> >-# Generate man page if asciidoc and xmlto are available.
> >-AC_ARG_VAR([XMLTO], [Path to xmlto command])
> >-AC_PATH_PROG([XMLTO], [xmlto])
> >-AC_ARG_VAR([ASCIIDOC], [Path to asciidoc command])
> >-AC_PATH_PROG([ASCIIDOC], [asciidoc])
> >-
> >-AM_CONDITIONAL([HAVE_DOCTOOLS], [test "x$XMLTO" != "x" && test "x$ASCIIDOC" != "x"])
> >+XORG_WITH_ASCIIDOC
> >+XORG_WITH_XMLTO
> >+AM_CONDITIONAL(HAVE_DOCTOOLS, [test "x$XMLTO" != "x" && test "x$ASCIIDOC" != "x"])
> > if test "x$XMLTO" = "x" || test "x$ASCIIDOC" = "x"; then
> > AC_MSG_WARN([xmlto or asciidoc not found - cannot create man pages without it])
> >-else
> >- if test x$MISC_MAN_SUFFIX = x ; then
> >- case $host_os in
> >- solaris*) MISC_MAN_SUFFIX=5 ;;
> >- *) MISC_MAN_SUFFIX=7 ;;
> >- esac
> >- fi
> >- AC_SUBST(MISC_MAN_SUFFIX)
> > fi
> >
> > AC_CONFIG_FILES([
> >
>
More information about the xorg-devel
mailing list