Bug #9821
Torbirdy uses Arabic as a default locale
100%
Description
I stumbled upon this bug while experimenting with Icedove in Tails. There is a 9 month old bug filed upstream at https://trac.torproject.org/projects/tor/ticket/13721.
More details can be found on the upstream ticket but to briefly summarize the problem: in a locale which is not supported/included by Torbirdy, Torbirdy’s UI will be displayed in Arabic. This is the case even when the corresponding language pack for Icedove is installed. Of the locales that Tails supports, I was able to reproduce this bug in FA, VN, and ZH.
The Torbirdy supported locales include:
ar, ca, cs, da, de, el, en, en-GB, es, eu, fr, he, hu, id, it, ko, lv, nb, nl, pa, pl, pt, pt-BR, ro, ru, sk, sl, sr, sv, tr, uk
It’s probably safe to say that Arabic will be the default for any locale which is not included with Torbirdy.
Subtasks
Related issues
Blocked by Tails - |
Resolved | 2015-09-27 |
History
#1 Updated by kytv 2015-07-30 01:43:20
- Parent task set to
Feature #5663
#2 Updated by sajolida 2015-08-09 08:34:44
- related to #8668 added
#3 Updated by intrigeri 2015-08-12 05:13:12
- related to deleted (
#8668)
#4 Updated by intrigeri 2015-08-12 05:13:23
- blocks #8668 added
#5 Updated by Anonymous 2015-09-27 05:15:24
- Assignee set to kytv
- Target version changed from 246 to Tails_1.7
- % Done changed from 0 to 10
- Type of work changed from Wait to Code
i’ve submitted a patch and pull request https://github.com/ioerror/torbirdy/pull/24.
kytv could you add that to the tpo bug please?
kytv fixes it as a delta in the meantime. assigning the ticket to him.
#6 Updated by kytv 2015-09-27 05:42:53
u wrote:
> i’ve submitted a patch and pull request https://github.com/ioerror/torbirdy/pull/24.
> kytv could you add that to the tpo bug please?
> kytv fixes it as a delta in the meantime. assigning the ticket to him.
I updated the upstream ticket to advise them of u ’s pull request.
#7 Updated by kytv 2015-09-27 06:02:50
- blocked by
Feature #10285: Add Icedove (and its related packages) to Tails added
#8 Updated by kytv 2015-09-27 06:17:23
- Status changed from Confirmed to In Progress
- % Done changed from 10 to 40
- Feature Branch set to kytv:bugfix/9821-torbirdy-arabic
In the event that upstream takes a while to fix this bug, the branch in this ticket will resolve this bug for Tails, allowing Icedove w/ torbirdy
to enter Tails at version 1.7.
Maybe we’ll luck out and this branch can be forgotten about.
#9 Updated by Anonymous 2015-09-28 02:11:32
our patch has been merged upstream: https://github.com/ioerror/torbirdy/commit/19cc1c71b9ee44554f3ff5ac6006da1380bd3a95
#10 Updated by kytv 2015-10-02 08:51:18
- Assignee changed from kytv to anonym
- QA Check set to Ready for QA
u wrote:
> our patch has been merged upstream: https://github.com/ioerror/torbirdy/commit/19cc1c71b9ee44554f3ff5ac6006da1380bd3a95
Excellent! Now we just need a new release with the fix or the Debian packages to include it, then the branch in this ticket can be discarded.
#11 Updated by kytv 2015-10-02 11:54:32
Debian bug: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=800706
#12 Updated by Anonymous 2015-10-02 13:12:07
kytv wrote:
> u wrote:
> > our patch has been merged upstream: https://github.com/ioerror/torbirdy/commit/19cc1c71b9ee44554f3ff5ac6006da1380bd3a95
>
> Excellent! Now we just need a new release with the fix or the Debian packages to include it, then the branch in this ticket can be discarded.
I’ve prepared the Debian package but I need to review the branch layout before it can be uploaded: https://lists.alioth.debian.org/pipermail/pkg-privacy-maintainers/Week-of-Mon-20150928/000038.html
#13 Updated by anonym 2015-10-05 01:46:23
- Assignee changed from anonym to kytv
- QA Check changed from Ready for QA to Dev Needed
#14 Updated by kytv 2015-10-05 02:32:32
- QA Check changed from Dev Needed to Ready for QA
#15 Updated by kytv 2015-10-05 02:35:44
- Assignee changed from kytv to anonym
#16 Updated by kytv 2015-10-05 07:11:32
Applied in changeset commit:f4d1f02dd965a8ebdc4a68048979d1c8c3a9d626.
#17 Updated by anonym 2015-10-07 08:16:18
- Assignee changed from anonym to kytv
- QA Check changed from Ready for QA to Pass
I’ve tested your local fix. However, let’s wait and see if we get a fixed Debian package first. Will you take care of that, kytv, since you’re already deep in the loop?
#18 Updated by Anonymous 2015-10-07 10:54:19
I’ve updated the package but need a sponsor to upload it, once the branch layout is fixed (doing that atm).
#19 Updated by Anonymous 2015-10-20 03:11:36
fyi thanks to intrigeri the updated package has now landed in Debian unstable.
#20 Updated by anonym 2015-10-22 07:43:17
Applied in changeset commit:c40f1b368e97397bb87ad4236d0dcb61eb68bedc.
#21 Updated by anonym 2015-10-22 07:43:18
- Status changed from In Progress to Resolved
- % Done changed from 40 to 100
Applied in changeset commit:1fb5c591b2f965d653380c0b9ad6d9bb8611ba1d.
#22 Updated by anonym 2015-10-22 07:44:40
- Assignee deleted (
kytv) - Feature Branch changed from kytv:bugfix/9821-torbirdy-arabic to feature/5663-return-to-icedove