Depends on lazr.restful

Bug #401990 reported by William Grant
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
lazr.restfulclient
Fix Released
Low
Unassigned

Bug Description

lazr.restfulclient depends on lazr.restful, but doesn't actually use it. lazr.restful depends on lots of Zope Toolkit bits, making launchpadlib's installation eat lots of bandwidth.

Revision history for this message
Leonard Richardson (leonardr) wrote :

To be precise, lazr.restfulclient only uses lazr.restful for its unit test suite.

Revision history for this message
Gary Poster (gary) wrote :

Edited IRC log:

[11:37am] gary_poster: We could easily have buildout require lazr.restful for tests, and then remove the core dependency. However, philosophically, that means we don't have any assurance that lazr.restfulclient really works without that dependency (and even its dependencies!). Therefore, I kind of hate that approach.
[11:38am] gary_poster: Perhaps there is a middle ground.
[11:38am] wgrant: gary_poster: Well, depending on lazr.restful brings in dozens and dozens of completely unnecessary ZTK.
[11:38am] gary_poster: An option I've done in the past is to define two test suites
[11:38am] leonardr: i was going to suggest something like that
[11:38am] gary_poster: One test suite verifies basic operation with only standard dependencies...
[11:38am] gary_poster: ok
[11:39am] gary_poster: yeah, I'd be fine with that
[11:39am] leonardr: we do have some tests in lazr.restfulclient that run against fake wsgi 'web services'
[11:39am] leonardr: they don't do anything, but they prove that basic lazr.restfulclient functionality works
[11:39am] gary_poster: I'd be much happier with that. then.

Changed in lazr.restfulclient:
status: New → Triaged
importance: Undecided → Low
Revision history for this message
Gary Poster (gary) wrote :

Assigning a low priority, but if it is easy to do (half day start to landing) as I expect, I'm +1 on slipping this in sometime soon.

Changed in lazr.restfulclient:
status: Triaged → Fix Committed
Changed in lazr.restfulclient:
milestone: none → 0.14.0
Changed in lazr.restfulclient:
status: Fix Committed → Fix Released
milestone: 0.14.0 → 0.10.0
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.