Bug #9821

Torbirdy uses Arabic as a default locale

Added by kytv 2015-07-30 01:41:45 . Updated 2015-10-22 07:44:40 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Target version:
Start date:
2015-07-30
Due date:
% Done:

100%

Feature Branch:
feature/5663-return-to-icedove
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Email Client
Deliverable for:
268

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 - Feature #10285: Add Icedove (and its related packages) to Tails Resolved 2015-09-27

History

#1 Updated by kytv 2015-07-30 01:43:20

#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.

#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.

#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