Closed
Bug 450964
Opened 16 years ago
Closed 16 years ago
Get rid of remaining SVG build options in comm-central
Categories
(MailNews Core :: Build Config, defect)
MailNews Core
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.9.1a2
People
(Reporter: kairo, Assigned: kairo)
References
Details
Attachments
(1 file)
(deleted),
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
The comm-central doesn't need to care about SVG. Investigating what we need to port from bug 450495 I realized that we have removed most of the SVG stuff already at checkin, but a few things remain that should still be cleaned up.
I could do this as part of bug 445141 but it might be better to do the whole task in smaller chunks.
The most problematic case is http://mxr.mozilla.org/comm-central/source/mail/installer/windows/packages-static#393 but with the removal of vast parts of this that we already did I'm not sure if it works correctly at the moment, and removing the ifdef doesn't change more than adding two bogus warnings at packaging time for current default Thunderbird builds.
Assignee | ||
Comment 1•16 years ago
|
||
This patch removes the rest of SVG definitions in comm-central. I was informed that bug 366126 enabled SVG in Thunderbird, so that ifdef is actually even more unneeded as default builds have it anyway.
Assignee | ||
Updated•16 years ago
|
Updated•16 years ago
|
Attachment #334179 -
Flags: review?(bugzilla) → review+
Assignee | ||
Comment 2•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment 3•16 years ago
|
||
Does this mean |ac_add_options --disable-svg| support is somehow "reduced" ?
Target Milestone: --- → seamonkey2.0alpha
Comment 4•16 years ago
|
||
(In reply to comment #3)
> Does this mean |ac_add_options --disable-svg| support is somehow "reduced" ?
No. Options will still get passed to the core mozilla configure. All it reduces is the extra time to configure comm-central (which isn't much in this case).
Assignee | ||
Updated•16 years ago
|
Product: SeaMonkey → MailNews Core
QA Contact: build-config → build-config
Target Milestone: seamonkey2.0alpha → ---
Updated•16 years ago
|
Target Milestone: --- → mozilla1.9.1a2
Comment 5•15 years ago
|
||
(In reply to comment #4)
Actually, it looks like it does ... because SeaMonkey was missing some related #ifdef:
I filed bug 541267.
Flags: in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•