Stop building man pages by default
From pretty much the beginning [1], pbr has defaulted to building both man page and html output, but has failed to document it anywhere. People tend to copy-paste their 'setup.py' and 'conf.py', or rely on the 'cookiecutter' project, with very little understanding of what's going on under the hood (and why would you care - it's docs :)). This means that the vast majority of folks using 'pbr' (basically everyone in OpenStack) have been unwittingly building "man pages" as part of their doc builds for no good reason, which has also led to a lot of confusion when this magic behavior is the cause of bugs [2][3]. There's no good reason that pbr should default to building both man pages and html output. For folks that want this functionality, we should document it so they can use it. For everyone else though, let's do the sane thing and output html like the standard 'build_sphinx' plugin. [1] https://github.com/openstack-dev/pbr/commit/5b8b7f1d [2] https://bugs.launchpad.net/pbr/+bug/1681983 [3] https://bugs.launchpad.net/oslotest/+bug/1379998 Change-Id: I579134a2b7980669180c1666503b848835cc2957 Closes-Bug: #1681983
This commit is contained in:
parent
54fb6e71b7
commit
d4e4efd779
|
@ -380,6 +380,12 @@ documentation`__. In addition, the ``autodoc_index_modules``,
|
|||
``autodoc_tree_excludes`` options in the ``pbr`` section will affect the output
|
||||
of the automatic module documentation generation.
|
||||
|
||||
.. versionchanged:: 3.0
|
||||
|
||||
The ``build_sphinx`` plugin used to default to building both HTML and man
|
||||
page output. This is no longer the case, and you should explicitly set
|
||||
``builders`` to ``html man`` if you wish to retain this behavior.
|
||||
|
||||
__ http://www.sphinx-doc.org/en/stable/man/sphinx-apidoc.html
|
||||
__ http://www.sphinx-doc.org/en/stable/setuptools.html
|
||||
|
||||
|
|
|
@ -63,7 +63,7 @@ def _find_modules(arg, dirname, files):
|
|||
|
||||
class LocalBuildDoc(setup_command.BuildDoc):
|
||||
|
||||
builders = ['html', 'man']
|
||||
builders = ['html']
|
||||
command_name = 'build_sphinx'
|
||||
sphinx_initialized = False
|
||||
|
||||
|
@ -142,10 +142,6 @@ class LocalBuildDoc(setup_command.BuildDoc):
|
|||
self.builder_target_dir, self.doctree_dir,
|
||||
self.builder, confoverrides, status_stream,
|
||||
freshenv=self.fresh_env, warningiserror=self.warning_is_error)
|
||||
sphinx_config = app.config
|
||||
if self.builder == 'man' and len(
|
||||
getattr(sphinx_config, 'man_pages', '')) == 0:
|
||||
return
|
||||
self.sphinx_initialized = True
|
||||
|
||||
try:
|
||||
|
|
|
@ -308,9 +308,8 @@ class BuildSphinxTest(BaseSphinxTest):
|
|||
build_doc = packaging.LocalBuildDoc(self.distr)
|
||||
build_doc.finalize_options()
|
||||
|
||||
self.assertEqual(2, len(build_doc.builders))
|
||||
self.assertEqual(1, len(build_doc.builders))
|
||||
self.assertIn('html', build_doc.builders)
|
||||
self.assertIn('man', build_doc.builders)
|
||||
|
||||
build_doc = packaging.LocalBuildDoc(self.distr)
|
||||
build_doc.builders = ''
|
||||
|
|
Loading…
Reference in New Issue