Merge lp:~salgado/launchpad/update-pygpgme into lp:launchpad
Status: | Merged |
---|---|
Approved by: | Brad Crittenden |
Approved revision: | no longer in the source branch. |
Merged at revision: | not available |
Proposed branch: | lp:~salgado/launchpad/update-pygpgme |
Merge into: | lp:launchpad |
Diff against target: |
249 lines (+31/-35) 8 files modified
lib/canonical/launchpad/doc/emailauthentication.txt (+1/-1) lib/canonical/launchpad/doc/gpghandler.txt (+6/-6) lib/canonical/launchpad/utilities/gpghandler.py (+2/-2) lib/lp/archivepublisher/tests/archive-signing.txt (+1/-2) lib/lp/archiveuploader/tests/nascentuploadfile.txt (+4/-4) lib/lp/registry/doc/gpg-signatures.txt (+14/-13) lib/lp/soyuz/doc/soyuz-upload.txt (+2/-6) utilities/sourcedeps.conf (+1/-1) |
To merge this branch: | bzr merge lp:~salgado/launchpad/update-pygpgme |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Brad Crittenden (community) | code | Approve | |
Review via email: mp+23790@code.launchpad.net |
Description of the change
Use the latest pygpgme, in preparation for switching to python2.6.
The last pygpgme revision (r49) includes a merge from trunk, which
brings http://
that was responsible for turning gpgmekey.fpr into a unicode. That
exposed a bug in our code where we were not ascii-encoding the
fingerprint before passing it to gpgme.Context.
callsites on that module do.
Another alternative here would be to change PymeKey.
to store the ascii-encoded fingerprint, instead of the unicode one, in
an instance variable. I'd be in favour of this, I think.
= Launchpad lint =
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
Linting changed files:
lib/canonical
lib/lp/
lib/lp/
lib/canonical
lib/canonical
lib/lp/
utilities/
== Pylint notices ==
lib/canonical/
25: [F0401] Unable to import 'gpgme'
26: [F0401] Unable to import 'gpgme'
30: [F0401] Unable to import 'canonical.config'
31: [F0401] Unable to import 'lp.registry.
33: [F0401] Unable to import 'canonical.
37: [F0401] Unable to import 'canonical.
Thanks for the fix Salgado.
To get around the unicode marker in test output we decided to do:
>>> print name
Salgado
instead of
>>> name
u'Salgado'
I think that is less fragile. Would you make the change? Of course you can't do anything about the unicode inside the exceptions.