Commit 3a6b96c7 authored by Stephen Finucane's avatar Stephen Finucane Committed by Stephen Finucane

Use 'build_reno' setuptools extension if available

reno recently gained a 'build_reno' command that could be used to build
a reno cache file via setuptools. Integrate this and start including
that file as part of the final distribution.

Change-Id: I54cb7a894770040165cc918469cb0ed486518789
parent 3c059cb7
......@@ -532,11 +532,57 @@ class LocalSDist(sdist.sdist):
command_name = 'sdist'
def checking_reno(self):
"""Ensure reno is installed and configured.
We can't run reno-based commands if reno isn't installed/available, and
don't want to if the user isn't using it.
if hasattr(self, '_has_reno'):
return self._has_reno
# versions of reno witout this module will not have the required
# feature, hence the import
from reno import setup_command # noqa
except ImportError:'[pbr] reno was not found or is too old. Skipping '
'release notes')
self._has_reno = False
return False
conf, output_file, cache_file = setup_command.load_config(
if not os.path.exists(os.path.join(conf.reporoot, conf.notespath)):'[pbr] reno does not appear to be configured. Skipping '
'release notes')
self._has_reno = False
return False
self._files = [output_file, cache_file]'[pbr] Generating release notes')
self._has_reno = True
return True
sub_commands = [('build_reno', checking_reno)] + sdist.sdist.sub_commands
def run(self):
# sdist.sdist is an old style class, can't use super()
def make_distribution(self):
# This is included in make_distribution because setuptools doesn't use
# 'get_file_list'. As such, this is the only hook point that runs after
# the commands in 'sub_commands'
if self.checking_reno():
from pbr import builddoc
_have_sphinx = True
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment