Closed Bug 722251 Opened 13 years ago Closed 11 years ago

Calendar should link .xpt files on all platforms

Categories

(Calendar :: Build Config, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: sgautherie, Unassigned)

References

Details

While checking bug 717493, I noticed that Calendar links .xpt files on (Opt) OS X64 builds only.
Fwiw, SeaMonkey always does it.

Is there a reason not to do it on all platforms? (And for Debug too, if that applies.)

NB: In build logs, search for "Linking .xpt files...".

http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/installer/xptlink.pl#97
Do you refer to building Lightning or Sunbird?
(In reply to Stefan Sitter from comment #1)
> Do you refer to building Lightning or Sunbird?

Lightning, from
http://build.mozillamessaging.com/tinderboxpushlog/?tree=CalendarTrunk
Whilst Lightning could link its xpt files (there are four of them in the shipped files), the command you are referencing is actually from the Thunderbird part of the log, not the Lightning specific bit, so afaict none of these currently get linked together.

I suspect when Lightning moves to being a non-binary extension, then these will probably just go away, or be zipped up anyway, so it won't actually matter then.
Severity: minor → enhancement
I can't seem to find "Linking .xpt files..." anywhere in the current logs, it seems this is no longer being done? As we are indeed moving towards ical.js I don't think we should do anything special unless this has major impact. If its an easy fix, feel free to reopen with a patch.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.