Closed
Bug 722251
Opened 13 years ago
Closed 11 years ago
Calendar should link .xpt files on all platforms
Categories
(Calendar :: Build Config, enhancement)
Calendar
Build Config
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
Comment 1•13 years ago
|
||
Do you refer to building Lightning or Sunbird?
Reporter | ||
Comment 2•13 years ago
|
||
(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
Comment 3•13 years ago
|
||
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
Comment 4•11 years ago
|
||
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.
Description
•