882307 - Add en-GB locale to the Fennec multi-locale build

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Karen Rudnitski [:kar]]

Looking to get an en-GB locale stood up to enable more refined, locale-based default customizations.

Let's talk about timing once you've had a chance to see about getting this organized.

Comment 1

[Axel Hecht [:Pike]]

The en-GB community opened up bug 689253 a while ago, let's track the some of the status over there.

Comment 2

[Axel Hecht [:Pike]]

Attachment: add en-GB to the single-locale builds

Mark, we have traction with the localizer for en-GB.

I suggest we add en-GB to the single-locale builds now, and to the multi-locale builds once 24 hits aurora.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 761688 [details] [diff] [review]
add en-GB to the single-locale builds

Sounds like a good plan

Comment 4

[Axel Hecht [:Pike]]

Comment on attachment 761688 [details] [diff] [review]
add en-GB to the single-locale builds

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: 
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky): 
String or IDL/UUID changes made by this patch:

Requesting builds for single locale for en-GB, no impact on the multi-locale build, thus no risk for the mobile product, and doesn't affect desktop at all.

Comment 5

[Alex Keybl [:akeybl]]

Presumably this is a very small increase in APK size because of l10n overlap. Can you confirm?

Comment 6

[Axel Hecht [:Pike]]

Yes.

Comment 7

[Axel Hecht [:Pike]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/92c47d1a801d.

I'm leaving this open for the actual multi-locale work on 24, and marking as status-firefox23: fixed for the single-locale work. I hope that works with the tracking queries.

Comment 8

[Jeff Beatty [:gueroJeff]]

We need to get en-GB added to mobile/android/locales/maemo-locales for Fx 24. Adding appropriate flags to have the work done.

Comment 9

[Jeff Beatty [:gueroJeff]]

Attachment: add-enGB.patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: 
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky): 
String or IDL/UUID changes made by this patch:

Comment 10

[Axel Hecht [:Pike]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/8683df1543d4

Comment 11

[u421692]

Test Case created in Moztrap:
https://moztrap.mozilla.org/manage/case/9388/