Skip to content

Commit

Permalink
man: enable warning msgs for "all" target
Browse files Browse the repository at this point in the history
* and remove `local-check` target. because we have killed all warnings,
  and `make check` depends on `am-all` actually. so previously, `make check`
  basically does nothing at all for validating the man pages' syntax.
  `make all` has rendered them already.
* do not run sphinx-build if $(top_srcdir)/doc/man does not exist. this
  is exactly the case of `make install` from a dist tarball.
* remove generated man pages in `make maintainer-clean` not in `make
  clean`, because we don't ship .rst files in the dist tarball.

Signed-off-by: Kefu Chai <[email protected]>
  • Loading branch information
tchaikov committed Apr 7, 2015
1 parent 251ae61 commit 1926bb9
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions man/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -12,17 +12,18 @@ if ENABLE_SERVER
include Makefile-server.am
endif

check-local:
${SPHINX_BUILD} -b man -d doctrees -c . $(top_srcdir)/doc/man $(top_builddir)/man

# prevent `make` from running in parallel, sphinx runs better in batch mode.
.PHONY: sphinx-build.stamp

$(dist_man_MANS): sphinx-build.stamp

# in a tree populated from dist tarball, the $(top_srcdir)/doc is not included
sphinx-build.stamp:
${SPHINX_BUILD} -Q -b man -d doctrees -c . $(top_srcdir)/doc/man $(top_builddir)/man
if [ -d $(top_srcdir)/doc/man ] ; then \
${SPHINX_BUILD} -b man -d doctrees -c . $(top_srcdir)/doc/man $(top_builddir)/man; \
fi

clean-local:
@rm -f *.8
@rm -rf doctrees

MAINTAINERCLEANFILES = $(dist_man_MANS)

0 comments on commit 1926bb9

Please sign in to comment.