Restore xh locale in maemo-locale for multi-locale builds

RESOLVED FIXED in Firefox 48

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: flod, Assigned: delphine)

Tracking

48 Branch
Firefox 48
All
Android
Points:
---

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Locale was removed in bug 1207659 but never added back

Per email discussion inside l10n-drivers, we should add it back.

@delphine
Let me know if you have time to work on this before traveling, otherwise I'll take it.
I'll take this now. Thanks
Attachment #8758024 - Flags: review?(francesco.lodolo)
(Reporter)

Comment 3

3 years ago
Comment on attachment 8758024 [details] [diff] [review]
bug1276674.patch

Review of attachment 8758024 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, thanks Delphine.
Attachment #8758024 - Flags: review?(francesco.lodolo) → review+
Comment on attachment 8758024 [details] [diff] [review]
bug1276674.patch

Please land directly on mozilla aurora, not central. thanks!
Attachment #8758024 - Flags: approval-mozilla-aurora?
Comment on attachment 8758024 [details] [diff] [review]
bug1276674.patch

Should we relnote it?
Attachment #8758024 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
(In reply to Sylvestre Ledru [:sylvestre] from comment #5)
> Comment on attachment 8758024 [details] [diff] [review]
> bug1276674.patch
> 
> Should we relnote it?

No relnotes needed since xh used to already be there - and we didn't relnote the removal. So no need to do anything at this point. Thanks :)
You need to log in before you can comment on or make changes to this bug.