PBR

pbr is failing to parse versions

Bug #1361251 reported by Flavio Percoco on 2014-08-25

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	PBR	Fix Released	Critical	Robert Collins

Bug Description

After tagging glance.store 0.0.1a2, this error was raised:

2014-08-25 11:03:21.216 | Syncing /opt/stack/new/glance.store/test-requirements.txt
2014-08-25 11:03:21.263 | Syncing setup.py
2014-08-25 11:03:21.279 | + setup_package /opt/stack/new/glance.store -e
2014-08-25 11:03:21.279 | + local project_dir=/opt/stack/new/glance.store
2014-08-25 11:03:21.279 | + local flags=-e
2014-08-25 11:03:21.279 | + pip_install -e /opt/stack/new/glance.store
2014-08-25 11:03:21.287 | + sudo PIP_DOWNLOAD_CACHE=/var/cache/pip http_proxy= https_proxy= no_proxy= /usr/local/bin/pip install --build=/tmp/pip-build.IZ7j7 -e /opt/stack/new/glance.store
2014-08-25 11:03:21.584 | DEPRECATION: --no-install, --no-download, --build, and --no-clean are deprecated. See https://github.com/pypa/pip/issues/906.
2014-08-25 11:03:21.588 | Obtaining file:///opt/stack/new/glance.store
2014-08-25 11:03:21.669 | Running setup.py (path:/opt/stack/new/glance.store/setup.py) egg_info for package from file:///opt/stack/new/glance.store
2014-08-25 11:03:22.109 | ERROR:root:Error parsing
2014-08-25 11:03:22.109 | Traceback (most recent call last):
2014-08-25 11:03:22.109 | File "/usr/local/lib/python2.7/dist-packages/pbr/core.py", line 104, in pbr
2014-08-25 11:03:22.109 | attrs = util.cfg_to_args(path)
2014-08-25 11:03:22.110 | File "/usr/local/lib/python2.7/dist-packages/pbr/util.py", line 238, in cfg_to_args
2014-08-25 11:03:22.110 | pbr.hooks.setup_hook(config)
2014-08-25 11:03:22.110 | File "/usr/local/lib/python2.7/dist-packages/pbr/hooks/__init__.py", line 27, in setup_hook
2014-08-25 11:03:22.110 | metadata_config.run()
2014-08-25 11:03:22.110 | File "/usr/local/lib/python2.7/dist-packages/pbr/hooks/base.py", line 29, in run
2014-08-25 11:03:22.110 | self.hook()
2014-08-25 11:03:22.110 | File "/usr/local/lib/python2.7/dist-packages/pbr/hooks/metadata.py", line 28, in hook
2014-08-25 11:03:22.110 | self.config['name'], self.config.get('version', None))
2014-08-25 11:03:22.111 | File "/usr/local/lib/python2.7/dist-packages/pbr/packaging.py", line 895, in get_version
2014-08-25 11:03:22.111 | version = _get_version_from_git(pre_version)
2014-08-25 11:03:22.111 | File "/usr/local/lib/python2.7/dist-packages/pbr/packaging.py", line 844, in _get_version_from_git
2014-08-25 11:03:22.111 | semver = version.SemanticVersion.from_pip_string(last_version)
2014-08-25 11:03:22.111 | File "/usr/local/lib/python2.7/dist-packages/pbr/version.py", line 221, in from_pip_string
2014-08-25 11:03:22.111 | raise ValueError('Unknown remainder %r' % (remainder,))
2014-08-25 11:03:22.111 | ValueError: Unknown remainder [u'1a2']
2014-08-25 11:03:22.111 | error in setup command: Error parsing /opt/stack/new/glance.store/setup.cfg: ValueError: Unknown remainder [u'1a2']
2014-08-25 11:03:22.161 | Complete output from command python setup.py egg_info:
2014-08-25 11:03:22.161 | ERROR:root:Error parsing
2014-08-25 11:03:22.161 |
2014-08-25 11:03:22.161 | Traceback (most recent call last):
2014-08-25 11:03:22.161 |
2014-08-25 11:03:22.161 | File "/usr/local/lib/python2.7/dist-packages/pbr/core.py", line 104, in pbr
2014-08-25 11:03:22.161 |
2014-08-25 11:03:22.161 | attrs = util.cfg_to_args(path)
2014-08-25 11:03:22.161 |
2014-08-25 11:03:22.161 | File "/usr/local/lib/python2.7/dist-packages/pbr/util.py", line 238, in cfg_to_args
2014-08-25 11:03:22.161 |
2014-08-25 11:03:22.161 | pbr.hooks.setup_hook(config)
2014-08-25 11:03:22.161 |
2014-08-25 11:03:22.161 | File "/usr/local/lib/python2.7/dist-packages/pbr/hooks/__init__.py", line 27, in setup_hook
2014-08-25 11:03:22.161 |
2014-08-25 11:03:22.162 | metadata_config.run()
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | File "/usr/local/lib/python2.7/dist-packages/pbr/hooks/base.py", line 29, in run
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | self.hook()
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | File "/usr/local/lib/python2.7/dist-packages/pbr/hooks/metadata.py", line 28, in hook
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | self.config['name'], self.config.get('version', None))
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | File "/usr/local/lib/python2.7/dist-packages/pbr/packaging.py", line 895, in get_version
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | version = _get_version_from_git(pre_version)
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | File "/usr/local/lib/python2.7/dist-packages/pbr/packaging.py", line 844, in _get_version_from_git
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | semver = version.SemanticVersion.from_pip_string(last_version)
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | File "/usr/local/lib/python2.7/dist-packages/pbr/version.py", line 221, in from_pip_string
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | raise ValueError('Unknown remainder %r' % (remainder,))
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | ValueError: Unknown remainder [u'1a2']
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | error in setup command: Error parsing /opt/stack/new/glance.store/setup.cfg: ValueError: Unknown remainder [u'1a2']
2014-08-25 11:03:22.162 |
2014-08-25 11:03:22.162 | ----------------------------------------

Since the version is wrong - the first 3 parts of the version should be just numbers - we should just report the error when the sdist is created to avoid having issues installing it.