diff options
author | Jenkins <jenkins@review.openstack.org> | 2016-10-02 01:47:52 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2016-10-02 01:47:53 +0000 |
commit | 2ccb2d21c0fa1f7c1b5647f135e009988be5ef92 (patch) | |
tree | 269ef1c713eefb32b9b779d8c1e819057e27502b /pbr | |
parent | 0a007e9091408b4aa327d7f3cf09e868f38117e0 (diff) | |
parent | a432bc2930ad0c5463163654bc18a18f8e2b417e (diff) | |
download | pbr-2ccb2d21c0fa1f7c1b5647f135e009988be5ef92.tar.gz |
Merge "Don't ignore data-files"
Diffstat (limited to 'pbr')
-rw-r--r-- | pbr/tests/test_packaging.py | 11 | ||||
-rw-r--r-- | pbr/tests/testpackage/setup.cfg | 2 | ||||
-rw-r--r-- | pbr/util.py | 6 |
3 files changed, 15 insertions, 4 deletions
diff --git a/pbr/tests/test_packaging.py b/pbr/tests/test_packaging.py index f532b76..b84cc9a 100644 --- a/pbr/tests/test_packaging.py +++ b/pbr/tests/test_packaging.py @@ -304,6 +304,17 @@ class TestPackagingInGitRepoWithCommit(base.BaseTestCase): self.expectThat(stdout, matchers.Contains('Generating ChangeLog')) +class TestExtrafileInstallation(base.BaseTestCase): + def test_install_glob(self): + stdout, _, _ = self.run_setup( + 'install', '--root', self.temp_dir + 'installed', + allow_fail=False) + self.expectThat( + stdout, matchers.Contains('copying data_files/a.txt')) + self.expectThat( + stdout, matchers.Contains('copying data_files/b.txt')) + + class TestPackagingInGitRepoWithoutCommit(base.BaseTestCase): def setUp(self): diff --git a/pbr/tests/testpackage/setup.cfg b/pbr/tests/testpackage/setup.cfg index c4ba378..a6d127a 100644 --- a/pbr/tests/testpackage/setup.cfg +++ b/pbr/tests/testpackage/setup.cfg @@ -30,7 +30,7 @@ keywords = packaging, distutils, setuptools [files] packages = pbr_testpackage package-data = testpackage = package_data/*.txt -data-files = testpackage/data_files = data_files/*.txt +data-files = testpackage/data_files = data_files/* extra-files = extra-file.txt [entry_points] diff --git a/pbr/util.py b/pbr/util.py index def3117..22aa06e 100644 --- a/pbr/util.py +++ b/pbr/util.py @@ -105,7 +105,7 @@ D1_D2_SETUP_ARGS = { "description": ("metadata", "summary"), "keywords": ("metadata",), "long_description": ("metadata", "description"), - "download-url": ("metadata",), + "download_url": ("metadata",), "classifiers": ("metadata", "classifier"), "platforms": ("metadata", "platform"), # ** "license": ("metadata",), @@ -215,6 +215,8 @@ def cfg_to_args(path='setup.cfg', script_args=()): config = {} for section in parser.sections(): config[section] = dict(parser.items(section)) + for k in config[section]: + config[section][k.replace('-', '_')] = config[section].pop(k) # Run setup_hooks, if configured setup_hooks = has_get_option(config, 'global', 'setup_hooks') @@ -652,8 +654,6 @@ def run_command_hooks(cmd_obj, hook_kind): def has_get_option(config, section, option): if section in config and option in config[section]: return config[section][option] - elif section in config and option.replace('_', '-') in config[section]: - return config[section][option.replace('_', '-')] else: return False |