107301 - Remove xpidl from xpcom module

Status: VERIFIED DUPLICATE of bug 107302

(SeaMonkey :: Build Config, defect)

Description

[hacker formerly known as [e-mail]]

xpidl should be split from the xpcom module and built from the toplevel
makefile.  It's only dependency is NSPR.  Since it is such an essential part of
the build, it is built as part of the export phase anyway.  I'm not sure whether
its worth actually moving it from the xpcom/ directory (I'll let dbradley make
that call) but it needs to be resorted in the build system for bug 105289.

Comment 1

[Brendan Eich [:brendan]]

I think xpidl should stay "near" the code it targets (typelibs, also .h files
that use xpcom to define interfaces).  That means under xpcom.  My 2 cents.

/be

Comment 2

[John Bandhauer]

It is also dependent on - and links to - libxpt (mozilla/xpcom/typelib/xpt) 
which is itself also linked into xpcom for use at runtime by the typelib loader.

Comment 3

[David Bradley]

I think it needs to stay in XPCOM as well. What is the issue with embedding? I
wasn't able to figure out why it needs to be resorted for embedding.

Comment 4

[hacker formerly known as [e-mail]]

The toplevel build system (at least) is being reorganized for embedding
purposes. The end goal is to have the embedding subset of the build be done in a
separate initial step and then build the rest of the app on top of it.  Moving
xpidl to be built from the toplevel Makefile (as opposed to being buried in
xpcom's heirarchy) is one of the initial "layering" goals.  This would be
similar to what we did with xpconnect & liveconnect in the unix build system but
for different reasons.

Comment 5

[Alec Flett]

Chris, are you imagining a situation where we have XPCOM but not xpidl? That
would be pretty cool, but we'd have to yank out xpconnect, libxpt, etc, as well.
I'm not sure how close we are in that area yet.. jband?

Comment 6

[Brendan Eich [:brendan]]

XPCOM without XPIDL is like a fish without a bicycle.  Truly, I think writing
C++-only bindings by hand is not "neat", and it's clearly a non-goal for any
project around here I've heard from, esp. mozilla1.0 -- please don't tell me
that embedders now want jus xpcom and not xpidl.

/be

Comment 7

[John Bandhauer]

Yeah, what brendan said (minus the fish and bicycle business :)

I understand wanting to clean up the build stuff, but I don't think that 
justifies moving the directories around.

Comment 8

[hacker formerly known as [e-mail]]

Next time I'll remember to phrase my request in the form of a question.  :)  It
was more of a "since we need to reorganize how things build for embedding, would
it make since to also move xpidl (actually typelib) out of xpcom and have xpcom
just be dependent upon xpidl(typelib) like it is upon nspr?"  Apparently, the
answer is no so I'll just change the build order and be on my way.

Comment 9

[hacker formerly known as [e-mail]]

aiee...s/make since/make sense/...where's my caffiene?!?!?

Comment 10

[hacker formerly known as [e-mail]]

I'm going to roll this into bug 107302.


*** This bug has been marked as a duplicate of 107302 ***

Comment 11

[Jon Granrose]

verified dup.