Closed Bug 1447641 Opened 2 years ago Closed 2 years ago
Clean up Dev
Tools l10n config
+++ 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.
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+
Pushed by email@example.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?
I'll do it. It would be nice if you could attach a patch :-)
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 firstname.lastname@example.org: 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: 2 years ago
Resolution: --- → FIXED
Both fixes worked out and the cross-channel automation works again.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.