[PATCH app-xfs 3/3] config: remove redundant MAKE_DEVEL_DOCS and option

Gaetan Nadon memsize at videotron.ca
Wed Jan 20 16:57:49 PST 2010


On Wed, 2010-01-20 at 11:30 -0800, Alan Coopersmith wrote:

> This was added because even if you have XMLTO you may not want to build
> the xfs internals documentation for developers which is unlikely to be
> of interest to 100% of people installing from packages.
> 

Plus the fact that some platforms (Gentoo OpenBSD) may have the tool
installed, but may be too old which breaks the build.
Those platforms asked for a generic macro where xmlto can be
unconditionally turned off the same way on all packages, rather than
reading custom code on each of them.

The option --without-xmlto will do the same. However, there may be a
case for a generic "don't build devel docs, which ever tool may be used"
option. This would be another util-macros feature to discuss. The server
has one, but, of course, it has a different name :-)

I don't have a problem in keeping it, that's why it is in a separate
patch. 


> 	-alan-
> 
> Gaetan Nadon wrote:
> > This is now handled by the generic XORG_WITH_XMLTO macro.
> > Platforms builder may need to adjust their build scripts.
> > 
> > Signed-off-by: Gaetan Nadon <memsize at videotron.ca>
> > ---
> >  Makefile.am  |    2 --
> >  configure.ac |    7 -------
> >  2 files changed, 0 insertions(+), 9 deletions(-)
> > 
> > diff --git a/Makefile.am b/Makefile.am
> > index 6d4d176..a3642f7 100644
> > --- a/Makefile.am
> > +++ b/Makefile.am
> > @@ -173,7 +173,6 @@ endif LINT
> >  # Developer documentation - not installed on system
> >  EXTRA_DIST += doc/xfs-design.xml doc/xfs-design.css doc/xfs-design.xsl
> >  
> > -if MAKE_DEVEL_DOCS
> >  if HAVE_XMLTO
> >  noinst_DATA = doc/xfs-design/index.html doc/xfs-design.txt
> >  CLEANFILES += $(noinst_DATA)
> > @@ -193,4 +192,3 @@ doc/xfs-design.txt: $(srcdir)/doc/xfs-design.xml $(srcdir)/doc/xfs-design.xsl
> >  	$(AM_V_GEN)$(XMLTO) -m $(srcdir)/doc/xfs-design.xsl -o doc txt $(srcdir)/doc/xfs-design.xml
> >  
> >  endif HAVE_XMLTO
> > -endif MAKE_DEVEL_DOCS
> > diff --git a/configure.ac b/configure.ac
> > index b429ed5..a0cfc44 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -95,13 +95,6 @@ AC_ARG_WITH(default-font-path, AS_HELP_STRING([--with-default-font-path=PATH], [
> >                                  [ FONTPATH="${DEFAULT_FONT_PATH}" ])
> >  AC_SUBST([FONTPATH])
> >  
> > -# Option to enable support for building developer internals docs
> > -AC_ARG_ENABLE(devel-docs,
> > -	AS_HELP_STRING([--enable-devel-docs],
> > -	    [Build internals documentation for developers (default: enabled)]),
> > -	    [DEVEL_DOCS=$enableval], [DEVEL_DOCS=yes])
> > -AM_CONDITIONAL([MAKE_DEVEL_DOCS], [test "x$DEVEL_DOCS" = "xyes"])
> > -
> >  # Option to enable support for starting from inetd
> >  AC_ARG_ENABLE(inetd,
> >  	AS_HELP_STRING([--enable-inetd],
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.x.org/archives/xorg-devel/attachments/20100120/b2a9c2f7/attachment.html 


More information about the xorg-devel mailing list