44092 - mail news is linking to rdf too much.

Status: VERIFIED WONTFIX

(MailNews Core :: Backend, defect, P3)

Description

[Judson Valeski]

See #43895 for more info.

Comment 1

[Judson Valeski]

+'ing

Comment 2

[Alec Flett]

*** Bug 44091 has been marked as a duplicate of this bug. ***

Comment 3

[Alec Flett]

the only places we should be linking against rdf are msgbaseutil and address 
book - Adding putterman to CC so he can verify my claim.

Also adding ducarroz to the CC because I will need his help with the mac

Comment 4

[scottputterman]

as long as it builds there's no reason why we need to do this.

Comment 5

[Judson Valeski]

I'm not sure how mail/news is built out, but we definately want to minimize 
these static links (on a module level) from a footprint/bloat standpoint. Does 
all of mail/news build into a single shared lib?

Comment 6

[scottputterman]

sorry, my comment was meant to be that as long as we build without linking 
against this library then making this change is fine.  I haven't tried it and 
for all I know that could be leftover from when we did need to link against it.

Comment 7

[Alec Flett]

ok, so it looks like either we have to figure out how to re-export nsRDFResource
from msgbaseutil.dll, or make msgbaseutil a static library itself (which I'm
beginning to think is The Right Thing anyway)

Comment 8

[Alec Flett]

jud this is turning out to be a whole lot more work than we thought at first.
I'd like to take this off the nsbeta2 train... the rdf library we're linking to
is VERY small (my DEBUG version is 41k) - we're not linking against RDF itself,
just a static utility library.

Comment 9

[Jean-Francois Ducarroz]

On Mac, we are not linking against RDF. However, we are including 
nsRDFResources.cpp directly into the Mac project and we are also linking against 
MsgUtil.lib (as static)

Comment 10

[Alec Flett]

ok, sounds like mac is doing the right thing.

Comment 11

[Judson Valeski]

mail news actually isn't even part of our current embedding solution, yanking
from beta2

Comment 12

[scottputterman]

moving to future.

Comment 13

[Alec Flett]

I'm marking WONTFIX. We can get past this by switching to resource delegates,
but I think putterman already has a bug on this.

Comment 14

[Håkan Waara]

Can someone verify this?

Comment 15

[[e-mail] (gone - use [e-mail] instead)]

AFAIK, the new outliner code doesn't use any (or much) RDF in the thread pane
(we still use it in the folder view).  But based on the comments in this bug,
verified.