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/features.py (+0/-1) breezy/tests/http_server.py (+0/-22) breezy/tests/https_server.py (+0/-14) breezy/tests/test_bzrdir.py (+0/-12) breezy/tests/test_http.py (+11/-169) breezy/transport/__init__.py (+4/-18) breezy/transport/http/__init__.py (+1/-1) breezy/transport/http/_pycurl.py (+0/-454) doc/en/release-notes/brz-3.0.txt (+4/-0) |
||||||||||||||||||||||||
To merge this branch: | bzr merge lp://staging/~jelmer/brz/kill-pycurl | ||||||||||||||||||||||||
Related bugs: |
|
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.
Looks good. If we want ntlm support again seems there are different python packages we could optional depend on instead.