450964 - Get rid of remaining SVG build options in comm-central

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Robert Kaiser]

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.

Comment 1

[Robert Kaiser]

Attachment: remove the rest of SVG definitions

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.

Comment 2

[Robert Kaiser]

Checked in as http://hg.mozilla.org/comm-central/index.cgi/rev/13eae6fc3733

Comment 3

[Serge Gautherie (:sgautherie)]

Does this mean |ac_add_options --disable-svg| support is somehow "reduced" ?

Comment 4

[Mark Banner (:standard8)]

(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).

Comment 5

[Serge Gautherie (:sgautherie)]

(In reply to comment #4)

Actually, it looks like it does ... because SeaMonkey was missing some related #ifdef:
I filed bug 541267.