Closed Bug 1779330 Opened 2 years ago Closed 2 years ago

Handle missing ftl files during L10n repackaging

Categories

(Thunderbird :: Build Config, enhancement)

enhancement

Tracking

(thunderbird_esr102+ fixed)

RESOLVED FIXED
105 Branch
Tracking Status
thunderbird_esr102 + fixed

People

(Reporter: rjl, Assigned: rjl)

References

Details

Attachments

(1 file)

Bug 1586984 documents an ongoing issue with localized builds. The short version is that incomplete locales are more incomplete at runtime than they need to be because one missing ftl file will cause a whole mess of strings to fallback to en-US.

Flod deals with this by periodically adding empty ftl files to the l10n-central repositories. He doesn't do that for Thunderbird strings unless someone asks though.

I'm proposing a change to l10n repackaging that would create these empty files (as part of "mach build installer-$AB_CD". No more manual intervention from the localization team is a good improvement.

Add a new mach command to add missing Fluent files in a localized build and
hook it in as part of "mach build installers-$AB_CD". This is to prevent Fluent
from falling back to en-US strings due to a missing localized strings file, and
will remove a manual process in the localization chain.

See bug 1586984.

The output logs when files are added.

Assignee: nobody → rob
Status: NEW → ASSIGNED
Attachment #9285225 - Attachment description: WIP: Bug 1779330 - Add missing ftl files during l10n packaging. → Bug 1779330 - Add missing ftl files during l10n packaging. r=#thunderbird-reviewers
Attachment #9285225 - Attachment description: Bug 1779330 - Add missing ftl files during l10n packaging. r=#thunderbird-reviewers → Bug 1779330 - Add missing ftl files during l10n packaging. r=dandarnell

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/01d457f9084d
Add missing ftl files during l10n packaging. r=dandarnell

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Blocks: 1786199
No longer blocks: 1786199
Target Milestone: --- → 105 Branch

Comment on attachment 9285225 [details]
Bug 1779330 - Add missing ftl files during l10n packaging. r=dandarnell

[Approval Request Comment]
Regression caused by (bug #): N/A
User impact if declined: Possible missing localized strings that should be translated
Testing completed (on c-c, etc.): been on c-c and c-c for a while with detected problems
Risk to taking this patch (and alternatives if risky):
This is my alternative to the ongoing l10n-central repo maintenance that is performed per bug 1586984. Since Thunderbird devs cannot write to the l10n-central repos to create these empty .ftl files, we have to rely on MoCo L10n team to do it. This hasn't been working out that well. This change creates the same empty .ftl files during L10n repackaging so that no changes to l10n-central repos are needed.
Uplifting this bug to c-esr102 will allow uplifting bug 574986 to c-esr102 without causing problems like bug 1786107.

Attachment #9285225 - Flags: approval-comm-esr102?

Comment on attachment 9285225 [details]
Bug 1779330 - Add missing ftl files during l10n packaging. r=dandarnell

[Triage Comment]
Approved for esr102

Attachment #9285225 - Flags: approval-comm-esr102? → approval-comm-esr102+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: