If folders in repository have unicode names, sometimes I get KeyError
Bug #869309 reported by
rsk
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Steve Kowalik | ||
| loggerhead |
Fix Committed
|
Critical
|
Unassigned | ||
| loggerhead-breezy |
Fix Released
|
Critical
|
Unassigned | ||
Bug Description
If folders in repository have unicode names, sometimes I get this error:
(root)/lenenergo : error
An unexpected error occurred whileprocessing the request:
KeyError: u'\u0428'
My patch to this bug attached.
Related branches
lp:~stevenk/launchpad/lh-480
- Steve Kowalik (community): Approve (code)
-
Diff: 34 lines (+4/-4)2 files modifiedutilities/sourcedeps.cache (+3/-3)
utilities/sourcedeps.conf (+1/-1)
Revision history for this message
| rsk (rsk7) wrote | #1 |
| summary: |
- If folder have unicode names, sometimes I get KeyError + If folders in repository have unicode names, sometimes I get KeyError |
| description: | updated |
| Changed in loggerhead: | |
| status: | New → Triaged |
| importance: | Undecided → Low |
| importance: | Low → High |
Revision history for this message
| Robert Collins (lifeless) wrote | #2 |
| tags: | added: trivial |
| Changed in loggerhead: | |
| importance: | High → Critical |
| tags: | added: oops |
| Changed in loggerhead: | |
| assignee: | nobody → john ford (kracker9219) |
| status: | Triaged → New |
| Changed in loggerhead: | |
| status: | New → Triaged |
| assignee: | john ford (kracker9219) → nobody |
| Changed in launchpad: | |
| status: | New → Triaged |
| importance: | Undecided → High |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #3 |
| Changed in launchpad: | |
| assignee: | nobody → Steve Kowalik (stevenk) |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | Triaged → Fix Committed |
| Changed in loggerhead: | |
| status: | Triaged → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
| Changed in loggerhead-breezy: | |
| status: | New → Fix Released |
| importance: | Undecided → Critical |
To post a comment you must log in.
Thanks for the patch, it looks sensible.