Closed Bug 1276674 Opened 4 years ago Closed 4 years ago

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

Categories

(Firefox for Android :: General, defect)

48 Branch
All
Android
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 48
Tracking Status
firefox48 --- fixed

People

(Reporter: flod, Assigned: delphine)

References

Details

Attachments

(1 file)

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
Attached patch bug1276674.patchSplinter Review
Attachment #8758024 - Flags: review?(francesco.lodolo)
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
Closed: 4 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.