Closed Bug 1447641 Opened 7 years ago Closed 7 years ago

Clean up DevTools l10n config

Categories

(Core :: Internationalization: Localization, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED

People

(Reporter: Pike, Assigned: Pike)

References

Details

Attachments

(2 files)

+++ This bug was initially created as a clone of Bug #1446222 +++ The DevTools landing in bug 1446222 changed the l10n.toml files in ways which make cross-channel unhappy. Notably, we need the config on central to be valid for all branches. We'll also need to port this to comm-central, https://hg.mozilla.org/comm-central/rev/008a5e87d53b copied the problem over.
I imagine you meant Bug 1444926? Updating the dependency.
Depends on: 1444926
No longer depends on: 1446222
Comment on attachment 8960932 [details] Bug 1447641, follow up to 1444926 to fix l10n.toml for cross-channel, https://reviewboard.mozilla.org/r/229678/#review235442 Nit on the commit message ::: commit-message-f4ddf:1 (Diff revision 1) > +Bug 1447641, follow up to 1446222 to fix l10n.toml for cross-channel, r?flod Need to update the bug number: "follow up to bug 1444926"
Attachment #8960932 - Flags: review?(francesco.lodolo) → review+
Severity: normal → major
Priority: P3 → P1
Pushed by axel@mozilla.com: https://hg.mozilla.org/mozilla-central/rev/7b55d395bb63 follow up to 1444926 to fix l10n.toml for cross-channel, r=flod, a=Aryx
Jorg, do you want me to land the corresponding fix in c-c, too, or are you going to do that?
Flags: needinfo?(jorgk)
I'll do it. It would be nice if you could attach a patch :-)
Flags: needinfo?(jorgk)
Attached patch comm-central patch (deleted) — Splinter Review
This is the patch, also see that I fixed a nit about '/' being in the one l10n path, which breaks stuff in subtle ways. That has been broken for long, never got around to fix it, now I do.
Thanks, I'll get this landed with the next batch soon.
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/2b766c459949 follow-up to bug 1444926 to fix l10n.toml for cross-channel (C-C part). r=flod
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Both fixes worked out and the cross-channel automation works again.
Status: RESOLVED → VERIFIED
Depends on: 1812367
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: