Removes temporary fix for doc generation

A temporary fix was added to get around a bug in how pbr handles
its autodoc_tree_index_modules setting. Since this bug is fixed we no
longer need the work around.

This change is needed since the workaround is now incompatible with the
latest version of Sphinx.

See also[1][2].

[1] https://review.opendev.org/191633
[2] https://review.opendev.org/121667

Change-Id: Idf9dc73d99b4233f49effe24aa047959b1a2ef2c
This commit is contained in:
Colleen Murphy 2019-07-05 16:38:15 -07:00
parent 8980f25545
commit ab2eb59068
4 changed files with 3 additions and 48 deletions

View File

View File

@ -1,41 +0,0 @@
# Copyright 2013 OpenStack Foundation
#
# Licensed under the Apache License, Version 2.0 (the "License"); you may
# not use this file except in compliance with the License. You may obtain
# a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations
# under the License.
import os.path as path
from sphinx import apidoc
# NOTE(gordc): pbr will run Sphinx multiple times when it generates
# documentation. Once for each builder. To run this extension we use the
# 'builder-inited' hook that fires at the beginning of a Sphinx build.
# We use ``run_already`` to make sure apidocs are only generated once
# even if Sphinx is run multiple times.
run_already = False
def run_apidoc(app):
global run_already
if run_already:
return
run_already = True
package_dir = path.abspath(path.join(app.srcdir, '..', '..', 'pycadf'))
source_dir = path.join(app.srcdir, 'api')
apidoc.main(['apidoc', package_dir, '-f',
'-H', 'pyCADF Modules',
'-o', source_dir])
def setup(app):
app.connect('builder-inited', run_apidoc)

View File

@ -10,10 +10,7 @@
# All configuration values have a default; values that are commented out
# serve to show the default.
import sys, os
# NOTE(gordc): path for Sphinx ext.apidoc
sys.path.insert(0, os.path.abspath('..'))
import os
# This is required for ReadTheDocs.org, but isn't a bad idea anyway.
os.environ['DJANGO_SETTINGS_MODULE'] = 'openstack_dashboard.settings'
@ -37,7 +34,6 @@ extensions = [
'sphinx.ext.coverage',
'sphinx.ext.viewcode',
'openstackdocstheme',
'ext.apidoc'
]
# Add any paths that contain templates here, relative to this directory.

View File

@ -47,5 +47,5 @@ upload-dir = doc/build/html
[pbr]
warnerrors = True
#autodoc_tree_index_modules = True
#autodoc_tree_root = ./pycadf
autodoc_tree_index_modules = True
autodoc_tree_root = ./pycadf