Closed Bug 1439469 Opened 2 years ago Closed Last year

Stop using INSTALL_EXTENSION_ID to build calendar.

Categories

(Thunderbird :: Build Config, enhancement)

enhancement
Not set

Tracking

(thunderbird_esr60 fixed, thunderbird60 fixed, thunderbird61 fixed)

RESOLVED FIXED
Thunderbird 61.0
Tracking Status
thunderbird_esr60 --- fixed
thunderbird60 --- fixed
thunderbird61 --- fixed

People

(Reporter: tomprince, Assigned: tomprince)

References

Details

Attachments

(1 file, 1 obsolete file)

This logic is going away, and it makes the build simpler as well.
Blocks: 1439265
Pushed by mozilla@hocat.ca:
https://hg.mozilla.org/comm-central/rev/8bbed059de0a
Package lightning directly as part of thunderbird; r=me
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
What is this actually doing?
Flags: needinfo?(mozilla)
This is getting rid of all the special case code to package an XPI for lightening, to then turn around and install all those files as part of thunderbird (part of that depends on makefile logic in m-c that is going away). Instead, this just installs the files directly into thunderbird where it will get picked up by the installer.

The primary reason for this is that the automation around packaging and L10n repacks in taskcluster isn't happy with the extra XPI being generated. In addition, since the code for handling lightening was unique, it created a maintenance burden as changes happened to the m-c L10n code, that needed to be ported to lightening.
Flags: needinfo?(mozilla)
I don't think it was a good idea to do this:
https://dxr.mozilla.org/comm-central/rev/8d636ad5810b5586433294c16cb2376d826fcca8/mail/locales/Makefile.in#97

Like this, we can't switch off Calendar any more in mail/config/mozconfigs/common :-(
Flags: needinfo?(mozilla)
Discussed on IRC, will fix it right now.
Flags: needinfo?(mozilla)
Target Milestone: --- → Thunderbird 61.0
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/28488be2f11c
Follow-up: Don't unconditionally reference calendar locales. r=tomprince (via IRC)
Attachment #8982597 - Attachment is obsolete: true
Jorg, this seems to have been uplifted to the beta TB 60 branch. Could you change the tracking flags if this hasn't been a mistake.

https://hg.mozilla.org/releases/comm-beta/rev/7cfa96f86122
Flags: needinfo?(jorgk)
Like so?
Flags: needinfo?(jorgk)
Thanks. Just wanted to be sure because I need to backport additional parts of bug 1451847 now.
Depends on: 1471326
To build langpacks for the lightning package I use:
make AB_CD=en-US L10N_XPI_NAME=lightning libs-$lang
That worked in 52, with this change applied, it no longer works. How can I build the langpacks for the lightning extension for the case we're building TB for the Linux distro (ie. we provide locales to the users without need of downloading them from mozilla)?
You need to log in before you can comment on or make changes to this bug.