Merge lp://qastaging/~fgiff/desktopcouch/527036 into lp://qastaging/desktopcouch
Proposed by
Frans Gifford
Status: | Merged |
---|---|
Merged at revision: | not available |
Proposed branch: | lp://qastaging/~fgiff/desktopcouch/527036 |
Merge into: | lp://qastaging/desktopcouch |
Diff against target: |
13 lines (+3/-0) 1 file modified
desktopcouch/local_files.py (+3/-0) |
To merge this branch: | bzr merge lp://qastaging/~fgiff/desktopcouch/527036 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Chad Miller (community) | Needs Fixing | ||
Review via email: mp+23264@code.qastaging.launchpad.net |
Commit message
Handle gnomekeyring.
Description of the change
Fix #527036 by handling exception where gnomekeyring dialog is cancelled.
To post a comment you must log in.
Hi Frans. Can you please add the same for all the locations where we access the keyring?
$ find . -name \*.py -type f -exec grep -l NoKeyringDaemon Error {} /dev/null \; tests/_ _init__ .py replication_ services/ ubuntuone. py local_files. py
./desktopcouch/
./desktopcouch/
./desktopcouch/
Add that and push again, and I will approve this outright and put it in the next release. There's not much time left!