Closed Bug 1905803 Opened 11 months ago Closed 10 months ago

Enable l10n strings from thunderbird-l10n

Categories

(Thunderbird :: Build Config, task)

Tracking

(thunderbird_esr115 fixed, thunderbird_esr128 fixed, thunderbird129 fixed)

RESOLVED FIXED
130 Branch
Tracking Status
thunderbird_esr115 --- fixed
thunderbird_esr128 --- fixed
thunderbird129 --- fixed

People

(Reporter: rjl, Assigned: rjl)

References

(Blocks 2 open bugs)

Details

Attachments

(7 files, 1 obsolete file)

Bug to actually use the strings in thunderbird-l10n.

See Also: → 1903349
Attachment #9410742 - Attachment description: WIP: Bug 1905803 - Use Github thunderbird-l10n for strings. r=#thunderbird-build-system-reviewers → Bug 1905803 - Use Github thunderbird-l10n for strings. r=#thunderbird-build-system-reviewers

Pushed by thunderbird@calypsoblue.org:
https://hg.mozilla.org/comm-central/rev/5a764a437807
Switch l10n_bumper to use thunderbird-l10n repo. r=dandarnell

Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Blocks: 1877804
Attachment #9416666 - Attachment is obsolete: true

Pushed by thunderbird@calypsoblue.org:
https://hg.mozilla.org/comm-central/rev/a471e335e982
Use Github thunderbird-l10n for strings. r=dandarnell
https://hg.mozilla.org/comm-central/rev/1a94c4c035fb
Enable l10n_bumper on comm central. r=dandarnell

Status: REOPENED → RESOLVED
Closed: 10 months ago10 months ago
Resolution: --- → FIXED

Comment on attachment 9416667 [details]
Bug 1905803 - Enable l10n_bumper on comm central. r=#thunderbird-build-system-reviewers

[Approval Request Comment]
L10n migration to Github. Requesting late beta uplift so that this is on comm-release post-merge on 2024-08-05.
Request is for 5a764a437807, a471e335e982, and 1a94c4c035fb.

Attachment #9416667 - Flags: approval-comm-beta?

Comment on attachment 9416667 [details]
Bug 1905803 - Enable l10n_bumper on comm central. r=#thunderbird-build-system-reviewers

[Approval Request Comment]
L10n move to Github. Not needed until 128.1.1+
Request is for 5a764a437807, a471e335e982, and 1a94c4c035fb.

[Approval Request Comment]
Needed for 115.15.0 - I will provide an esr115 patch prior to building.

Attachment #9416667 - Flags: approval-comm-esr128?
Attachment #9416667 - Flags: approval-comm-esr115?

Comment on attachment 9416667 [details]
Bug 1905803 - Enable l10n_bumper on comm central. r=#thunderbird-build-system-reviewers

[Triage Comment]
Approved for beta

Attachment #9416667 - Flags: approval-comm-esr128?
Attachment #9416667 - Flags: approval-comm-esr128+
Attachment #9416667 - Flags: approval-comm-esr115?
Attachment #9416667 - Flags: approval-comm-esr115+
Attachment #9416667 - Flags: approval-comm-beta?
Attachment #9416667 - Flags: approval-comm-beta+

Comment on attachment 9416667 [details]
Bug 1905803 - Enable l10n_bumper on comm central. r=#thunderbird-build-system-reviewers

Switching flags back to approval-comm-esr128? and approval-comm-esr115? so that we don't pick these up earlier than expected.

Attachment #9416667 - Flags: approval-comm-esr128?
Attachment #9416667 - Flags: approval-comm-esr128+
Attachment #9416667 - Flags: approval-comm-esr115?
Attachment #9416667 - Flags: approval-comm-esr115+
Target Milestone: --- → 130 Branch

[Approval Request Comment]
This patch combines the 3 patches on this bug as well as bug 1905802 and a final patch to l10n-changesets.json and is specific to comm-esr115.
This change has been on Daily for about a week and on 130.0b1 and 129.0rc monthly have been verified working via manual inspection.

Attachment #9417821 - Flags: approval-comm-esr115?
Attachment #9416667 - Flags: approval-comm-esr115?

[Approval Request Comment]
For comm-esr128, uplift from this bug and bug 1905802 the following changesets in this order:

  • 5a764a437807
  • 7b6cbb5340b0 (bug 1905802)
  • a471e335e982
  • 1a94c4c035fb

Then apply this patch to update l10n-changesets.json with a Git revision.

Attachment #9417822 - Flags: approval-comm-esr128?
Regressions: 1912126

Comment on attachment 9417821 [details] [diff] [review]
comm_esr115_l10n-bugs.patch

[Triage Comment]
Approved for esr115

Attachment #9417821 - Flags: approval-comm-esr115? → approval-comm-esr115+

Comment on attachment 9417822 [details] [diff] [review]
comm_esr128_l10n-changesets.patch

[Triage Comment]
Approved for esr128

Attachment #9417822 - Flags: approval-comm-esr128? → approval-comm-esr128+
Attachment #9416667 - Flags: approval-comm-esr128?

Is this supposed to bump l10n-changesets.json automatically on 128? That does not seem to be happening (128 still uses an old pinning)

Flags: needinfo?(rob)

(In reply to Magnus Melin [:mkmelin] from comment #20)

Is this supposed to bump l10n-changesets.json automatically on 128? That does not seem to be happening (128 still uses an old pinning)

It's not supposed to update on esr or release.

The logic behind that makes sense with a monthly stable release cadence -- as translators work through backlogs, they can see the results of their efforts in daily and beta. This never quite made sense for Thunderbird's yearly stable releases though.

I think this is a good idea though. It's a matter of when and how. I generally try to bump for minor versions for the first six months at least, but I'm a forgetful human...

Flags: needinfo?(rob)
Blocks: 1918457
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: