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.
Revision history for this message
| The Breezy Bot (the-breezy-bot) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Looks good. If we want ntlm support again seems there are different python packages we could optional depend on instead.