Closed Bug 1779047 Opened 3 years ago Closed 3 years ago

Adapt mach esmify for comm-central

Categories

(Thunderbird :: Upstream Synchronization, task)

Tracking

(thunderbird_esr102 unaffected, thunderbird107 fixed)

RESOLVED FIXED
107 Branch
Tracking Status
thunderbird_esr102 --- unaffected
thunderbird107 --- fixed

People

(Reporter: rjl, Assigned: rjl)

References

Details

Attachments

(1 file, 1 obsolete file)

mach esmify helps with esmification. As is, it probably isn't too useful for comm-central code.

Initial notes

The HGUtils class won't handle files in comm/ because it's a nested repo. It's inconvenient, but it's not a problem that hasn't been dealt with before. (Though completing comm-central one-repo project would help a lot here.)

Once HGUtils is hacked up, running ./mach esmify comm/mail completes, but with numerous warnings "Unknown module" warnings during rewrite_imports. That's probably because map.json doesn't list anything from comm-central.

Last, the eslint step needs some adjustment. When ESLint runs in CI, it is run from the comm/ sub-directory like ../mach eslint.

This won't work -- too much is relying on the javascript portion
living in /tools/esmify.

Attachment #9292077 - Attachment description: WIP: Bug 1779047 → WIP: Bug 1779047 - mach tb-esmify command. r=#thunderbird-reviewers

Depends on D155870

Depends on: 1792477
Assignee: nobody → rob
Status: NEW → ASSIGNED
Attachment #9292077 - Attachment description: WIP: Bug 1779047 - mach tb-esmify command. r=#thunderbird-reviewers → Bug 1779047 - mach tb-esmify command. r=#thunderbird-reviewers
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 107 Branch
Attachment #9296263 - Attachment is obsolete: true

Great job Rob, thanks for getting this going!

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: