Merge lp:~vila/bzr/2.2-693880-ssl-readline into lp:bzr/2.2
Status: | Merged |
---|---|
Approved by: | John A Meinel |
Approved revision: | no longer in the source branch. |
Merge reported by: | Vincent Ladeuil |
Merged at revision: | not available |
Proposed branch: | lp:~vila/bzr/2.2-693880-ssl-readline |
Merge into: | lp:bzr/2.2 |
Diff against target: |
46 lines (+16/-9) 2 files modified
NEWS (+3/-0) bzrlib/transport/http/_urllib2_wrappers.py (+13/-9) |
To merge this branch: | bzr merge lp:~vila/bzr/2.2-693880-ssl-readline |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
John A Meinel | Approve | ||
Jelmer Vernooij (community) | code | Approve | |
Review via email: mp+44671@code.launchpad.net |
Commit message
Fix https compatibility with python2.7
Description of the change
Bug #693880 broke the workaround describe in the patch (I won't copy it here :).
This came with the dev version that recently landed in natty.
Long story short, this is somewhat critical as it breaks https access, so the sooner we land it the better.
Since the patch is small, I targeted 2.2 so we won't have to backport in a hurry if people upgrade to python2.7 while using a stable version, but I think we may have released 2.3 when the corresponding change is released in python.
The most likely to encounter this problem are early natty adopters, so I'll make sure this get included in 2.3b5 asap and may release it on 2011-01-06 as previously planned.
I've tested this fix on babune again python 2.4, 2.5 and 2.7 and locally on 2.6.
Thanks for fixing this!
An inline comment explaining why this hack is necessary (and mentioning SSLFile) would be nice.