When doing a regular git build with xmlto missing, 'make all'
and 'make install' fails as it is trying to install man pages
that were not built.

The patch uses the same techinique used in libXtst and libXi.
However, this  introduces a new problem whereby man pages
are not created or installed when Xcomposite.man is present
(which is shipped in a tarball).

Both libraries work the same way now. The make dist will fail
when xmlto is missing unless Xcomposite.man is present. This is
what we want, not being able to create a crippled tarball.

Signed-off-by: Gaetan Nadon <[email protected]>
---
 man/Makefile.am |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/man/Makefile.am b/man/Makefile.am
index 13a75c5..2438efa 100644
--- a/man/Makefile.am
+++ b/man/Makefile.am
@@ -21,8 +21,10 @@ Xcomposite_man_aliases =                     \
 
 Xcomposite_shadowmen = $(Xcomposite_man_aliases:=...@lib_man_suffix@)
 
+if HAVE_XMLTO
 libman_DATA = $(libman_PRE:m...@lib_man_suffix@) \
              $(Xcomposite_shadowmen)
+endif
 
 EXTRA_DIST  = $(libman_PRE) $(libman_xml)
 
-- 
1.6.0.4

Initial post for review

Testing background:
Some platforms don't have xmlto, so they turn it off with --without-xmlto. 
When building from git, without xmlto:
  # it's not possible to create Xcomposite.man
  # it's not possible to create Xcomposite.3
  # 'make install' must not fail as libraries must be installed in lib
  # 'make dist' must fail as Xcomposite.man would be missing in the newly 
created tarball
When building from tarball, without xmlto, but with Xcomposite.man:
  # it's not possible to create Xcomposite.man, but it's already there 
(MAINTAINERCLEAN)
  # 'make install' is sucessful but no man pages are created or installed 
(that's the trade-off)
  # 'make dist' is successful, as Xcomposite.man is available

The complexity comes from the fact that Xcomposite.man is a generated 
intermediate source included in the tarball.
The platform may or may not have xmlto and it may or may not have 
Xcomposite.man (git or tarball)
When this platform is used to create tarballs, it must be allowed to create a 
tarball without Xcomposite.man.

The trade-off can be eliminated but it will increase the complexity of the 
makefile and introducing a new problem.

_______________________________________________
xorg-devel mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to