diff options
author | Jenkins <jenkins@review.openstack.org> | 2013-07-22 03:47:42 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2013-07-22 03:47:42 +0000 |
commit | db43d24cd55d6fc3d91c8955ccdcb3040074041c (patch) | |
tree | 097c7511ccf01a82031caf4305c1cdd3baeb4cdd | |
parent | 21390906eac2ba67789c7950b5f770feb76a433e (diff) | |
parent | c84876dc0f559a66fec19b2f81f5717204b253e2 (diff) | |
download | pbr-db43d24cd55d6fc3d91c8955ccdcb3040074041c.tar.gz |
Merge "Remove the need to specify the pbr hook"
-rw-r--r-- | README.rst | 3 | ||||
-rw-r--r-- | pbr/tests/__init__.py | 1 | ||||
-rw-r--r-- | pbr/util.py | 8 |
3 files changed, 8 insertions, 4 deletions
@@ -124,9 +124,6 @@ A simple sample can be found in pbr s own setup.cfg [files] packages = oslo - [hooks] - setup-hooks = - pbr.hooks.setup_hook The minimal setup.py should look something like this:: diff --git a/pbr/tests/__init__.py b/pbr/tests/__init__.py index 5109b0f..a4874c3 100644 --- a/pbr/tests/__init__.py +++ b/pbr/tests/__init__.py @@ -79,6 +79,7 @@ class BaseTestCase(testtools.TestCase, testresources.ResourcedTestCase): self.useFixture(fixtures.NestedTempfile()) self.useFixture(fixtures.FakeLogger()) + self.useFixture(fixtures.EnvironmentVariable('PBR_VERSION', '0.0')) self.temp_dir = self.useFixture(fixtures.TempDir()).path self.package_dir = os.path.join(self.temp_dir, 'testpackage') diff --git a/pbr/util.py b/pbr/util.py index 353635e..3faa58e 100644 --- a/pbr/util.py +++ b/pbr/util.py @@ -77,6 +77,7 @@ try: except ImportError: import ConfigParser as configparser +import pbr.hooks # A simplified RE for this; just checks that the line ends with version # predicates in () @@ -219,7 +220,9 @@ def cfg_to_args(path='setup.cfg'): try: if setup_hooks: - setup_hooks = split_multiline(setup_hooks) + setup_hooks = [ + hook for hook in split_multiline(setup_hooks) + if hook != 'pbr.hook.setup_hook'] for hook in setup_hooks: hook_fn = resolve_name(hook) try : @@ -233,6 +236,9 @@ def cfg_to_args(path='setup.cfg'): log.error(traceback.format_exc()) sys.exit(1) + # Run the pbr hook + pbr.hooks.setup_hook(config) + kwargs = setup_cfg_to_setup_kwargs(config) # Set default config overrides |