Declares that it requires the test dependencies, but doesn't depend on them, breaking other python code
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| lazr.restfulclient |
Triaged
|
High
|
Unassigned | ||
| lazr.restfulclient (Debian) |
Fix Released
|
Unknown
|
|||
| lazr.restfulclient (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
| Lucid |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
Impact: code that uses lazr.restfulclient (or launchpadlib) and pkg_resources can't work. The python package declares that the test dependencies are required, but they aren't listed in the .deb dependencies, and so aren't installed. They aren't needed for the code so they should be dropped.
Development fix: Drop the requires on the test dependencies
Stable fix: the same thing
Test Case:
1. Install the lazr.restfulclient package from -updates
2. Check that van.testing, wsgi_intercept, lazr.restful and lazr.authentication are listed in the first section of /usr/share/
3. Install the lazr.restfulclient package from -proposed
4. Check that those lines are now listed in a [tests] section.
Regression potential: low, as the packages haven't been installed as dependencies, and some aren't even packaged, so nothing should be relying on them being declared as dependencies.
| Launchpad Janitor (janitor) wrote | #1 |
| Changed in lazr.restfulclient (Ubuntu): | |
| status: | New → Fix Released |
| Stefano Rivera (stefanor) wrote | #2 |
| Changed in lazr.restfulclient (Debian): | |
| status: | Unknown → Fix Released |
| Changed in lazr.restfulclient: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| tags: | added: trivial |
| Martin Pitt (pitti) wrote Please test proposed package | #3 |
| Changed in lazr.restfulclient (Ubuntu Lucid): | |
| status: | New → Fix Committed |
| tags: | added: verification-needed |
| description: | updated |
| Bartosz Kosiorek (gang65) wrote | #4 |
| tags: |
added: verification-done removed: verification-needed |
| Michael Vogt (mvo) wrote | #5 |
| Launchpad Janitor (janitor) wrote | #6 |
| Changed in lazr.restfulclient (Ubuntu Lucid): | |
| status: | Fix Committed → Fix Released |
This bug was fixed in the package lazr.restfulclient - 0.12.0-1ubuntu1
---------------
lazr.restfulclient (0.12.0-1ubuntu1) precise; urgency=low
* Move test dependencies to extras_require so that setuptools doesn't think
they are needed for the code to work. (LP: #918307)
.
Having them listed in requires, but not in the package dependencies leads
to having a package installed that setuptools thinks is broken. This
breaks any code that relies on lazr.restfulclient (or launchpadlib) and
uses setuptools/
-- James Westby <email address hidden> Wed, 18 Jan 2012 12:50:39 -0500