Merge lp://staging/~jelmer/brz/kill-pycurl into lp://staging/brz

Proposed by Jelmer Vernooij
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: no longer in the source branch.
Merge reported by: The Breezy Bot
Merged at revision: not available
Proposed branch: lp://staging/~jelmer/brz/kill-pycurl
Merge into: lp://staging/brz
Diff against target: 991 lines (+20/-691)
9 files modified
breezy/tests/ (+0/-1)
breezy/tests/ (+0/-22)
breezy/tests/ (+0/-14)
breezy/tests/ (+0/-12)
breezy/tests/ (+11/-169)
breezy/transport/ (+4/-18)
breezy/transport/http/ (+1/-1)
breezy/transport/http/ (+0/-454)
doc/en/release-notes/brz-3.0.txt (+4/-0)
To merge this branch: bzr merge lp://staging/~jelmer/brz/kill-pycurl
Reviewer Review Type Date Requested Status
Martin Packman Approve
Review via email:

Commit message

Remove pycurl support.

Description of the change

Remove pycurl, go back to a single http implementation.

Remove the http+urllib:// bits, just use plain http URLs everywhere. This addresses several bugs.

To post a comment you must log in.
Revision history for this message
Martin Packman (gz) wrote :

Looks good. If we want ntlm support again seems there are different python packages we could optional depend on instead.

review: Approve
Revision history for this message
The Breezy Bot (the-breezy-bot) wrote :

A commit message must be set

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.


People subscribed via source and target branches