Closed Bug 1837269 Opened 2 years ago Closed 2 years ago

Port bug 1824612: Convert toolkit/components/extensions to ES modules

Categories

(Thunderbird :: Upstream Synchronization, task)

Thunderbird 116

Tracking

(thunderbird_esr102 unaffected, thunderbird115 fixed)

RESOLVED FIXED
116 Branch
Tracking Status
thunderbird_esr102 --- unaffected
thunderbird115 --- fixed

People

(Reporter: betterbird.proje.ct, Assigned: mkmelin)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1836657 +++

Bug 1824612 changed many Extension*.jsm to ES modules:
https://hg.mozilla.org/mozilla-central/rev/9c02dd904edb

These are still imported in TB:
https://searchfox.org/comm-central/search?q=gre%2Fmodules%2FExtension&path=&case=false&regexp=false

This should be fixed in TB and backported to 115 beta to avoid merge issues in the future. Note that bug 1824612 landed on Mozilla 115. Apparently some glue code is used to avoid failures in the consumers of the no-longer-existing JSM modules. Our project noticed since we were patching one of the JSM files which no longer exist.

Assignee: nobody → mkmelin+mozilla
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
See Also: → 1824612
Target Milestone: --- → 116 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/280d9dc96f26
Port bug 1824612: Convert toolkit/components/extensions to ES modules. r=leftmostcat

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED

Comment on attachment 9338234 [details]
Bug 1837269 - Port bug 1824612: Convert toolkit/components/extensions to ES modules. r=#thunderbird-reviewers

[Approval Request Comment]
Uplift to be on sync for what toolkit's using in 115. While fallbacks will work, there's room for confusion.

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

Comment on attachment 9338234 [details]
Bug 1837269 - Port bug 1824612: Convert toolkit/components/extensions to ES modules. r=#thunderbird-reviewers

[Triage Comment]
Approved for beta

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

Attachment

General

Created:
Updated:
Size: