543060 - remove fennec's dependency on xulrunner

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch

we seem to be moving away from the idea of having a useful system xulrunner, so let's not build it.

Comment 1

[(not currently active) Ted Mielczarek]

I noted this on IRC, but Firefox supports being built as a standalone app or --with-libxul-sdk, and Fennec should do the same.

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch v.2

based on ted's suggestion. This patch works both ways, based on --with-libxul-sdk being passed to configure.

Comment 3

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch v.3

we only export an import library for windows, os2 and wince bugs.

Comment 4

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch to fix installer

the 7zip installer assumes a few things are in dist/fennec/xulrunner, this fixes that assumption.

Also, I rearranged things a bit to make them readable

Comment 5

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch v.4

this runs on linux as long as you packages it (having fennec sym-linked to $(srcdir)/mobile/app/fennec makes us not find the application.ini because its in the objdir) and run it with:

./run-mozilla.sh ./fennec

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 428668 [details] [diff] [review]
patch v.4

Tested on Maemo and it built with and without the libxul-sdk.

if we decide to use this for Maemo, we'll need to update the fennec.desktop.in file so it is aware of the | ./run-mozilla.sh ./fennec | issue.

File as a followup bug?

Nice patch

Comment 7

[Brad Lassey [:blassey] (use needinfo?)]

pushed http://hg.mozilla.org/mobile-browser/rev/7227b5b43ebe