272783 - --enable-libxul

Status: RESOLVED FIXED

(Toolkit :: Startup and Profile System, defect)

Description

[Benjamin Smedberg]

Starting to get libxul building.

Comment 1

[Benjamin Smedberg]

Attachment: Configure bits only.

Here's what I'm thinking for the configure changes. This patch is useless
without an additional patch which sets LIBXUL_LIBRARY in some makefiles and
hook up the static build.

Comment 2

[cls]

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

Comment 3

[Benjamin Smedberg]

Attachment: Working patch

This patch builds and works. It should not affect the default build of anything
except xulrunner (I tested seamonkey and it doesn't break anything). It depends
on the other two bugs blocking this one.

Comment 4

[Benjamin Smedberg]

Comment on attachment 167796 [details] [diff] [review]
Working patch

Grr, I forgot -N

Comment 5

[Benjamin Smedberg]

Attachment: --enable-libxul with toolkit/libxul

Comment 6

[Benjamin Smedberg]

Comment on attachment 167839 [details] [diff] [review]
--enable-libxul with toolkit/libxul

Shaver, I've overloaded darin, so maybe you can look at this one?

Comment 7

[Darin Fisher]

Comment on attachment 167839 [details] [diff] [review]
--enable-libxul with toolkit/libxul

this patch looks really good to me.  my only comment was that we might want to
make the directory name be toolkit/library instead of toolkit/libxul since
there has been some discussion about the libxul name changing to something
else.

Comment 8

[Benjamin Smedberg]

Fixed on trunk. I'll open another bug for the additional tiers.