pbr/d2to1 should be build rather than runtime dependencies

Bug #1190539 reported by Stuart McLaren
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-swiftclient
In Progress
Undecided
Stuart McLaren

Bug Description

requirements.txt currently lists pbr and d2to1.

This leads to python-swiftclient having package requirements on python-pbr, python-d2to1, python-setuptools-git and git.

I'm guessing pbr and d2to1 should be in test-requirements.txt

Changed in python-swiftclient:
assignee: nobody → Stuart McLaren (stuart-mclaren)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-swiftclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/32862

Changed in python-swiftclient:
status: New → In Progress
Revision history for this message
Monty Taylor (mordred) wrote :

pbr/d2to1 are actually requirements to run setup.py at all, so you really do need them for everything. However, you should not need git as a dependency for debian packaging, as pbr has two different facilities to work in the absence of git information.

First, for the "normal" workflow, one runs python setup.py sdist in a git clone, and this produces a python source tarball. That tarball then contains the information that is needed about versioning and authors and whatnot to not git to be consumed. Then one uses that tarball as an input to the debian packaging process.

However, if the sdist-from-git-repo step is being skipped for some reason, there is an env var, OSLO_PACKAGE_VERSION, that you can set to declare the version and skip the git version discovery. Similarly, there are SKIP_WRITE_GIT_CHANGELOG and SKIP_GENERATE_AUTHORS env vars which you can set to skip making authors and changelog files from git.

Does that help? Are there any other issues you're running in to? pbr is basically a global requirement to always be installed for all of openstack now - so I'd like to fix issues in it whenever we have them. :)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.