734975 - Remove the XULRunner installer on Mac and switch the package to a .tar.bz2

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Benjamin Smedberg]

Currently we package a XULRunner installer on mac which will install XULRunner to /Library/Frameworks/XUL.framework. This wasn't a good idea to begin with and we removed support for even searching in system locations a while back. We want application authors to package XULRunner in their individual application bundles. To that end, I have removed the installer generation code and am just going "back" to packaging a .tar.bz2 file.

Comment 1

[Benjamin Smedberg]

Attachment: Remove the installer, rev. 1

Comment 2

[Dave Townsend [:mossop]]

Comment on attachment 605046 [details] [diff] [review]
Remove the installer, rev. 1

Review of attachment 605046 [details] [diff] [review]:
-----------------------------------------------------------------

This doesn't include the removals for xulrunner/installer/mac. Is that a mistake or intentional?

The readme looks good to me, I think as long as it links to the website then that should give everyone all the info they might want. Do we want to mention the license or is a LICENSE file already included somewhere?

Comment 3

[Benjamin Smedberg]

Attachment: Remove the installer, rev. 1.1

There is a separate LICENSE file. Not including the xulrunner/installer/mac removal in the patch was accidental. `hg rm` and `rm` aren't identical ;-)

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/6fd5e0d9d94b

Comment 5

[Benjamin Smedberg]

This broke the universal build due to flight.mk referencing xulrunner/installer/mac: I'll work on fixing that tomorrow.

Comment 6

[Dave Townsend [:mossop]]

Attachment: bustage fix WIP

This at least makes the build and package steps complete in universal builds. I haven't done any testing in non-UB or comparisons to what was generated before and after yet though.

Comment 7

[Dave Townsend [:mossop]]

Attachment: bustage fix

I've run builds locally in both non-universal and universal and all succeed and package both XULRunner and the SDK ok. I didn't go so far as comparing the exact files included because of one complication, it turns out that we now omnijar XULRunner where we didn't when doing a pkg. A brief check shows all relevant files are present and I can run an application with the generated framework.

There were a bunch of problems with packaging here, mostly stemming from the fact that all of the support for packaging a bundle is ifdeffed behind a check for the package format being DMG. I suspect it doesn't need to be ifdeffed behind anything but to minimise the potential for breakage elsewhere I hid it behind toolkit=cocoa here.

I also made a slight change to where we tar file from so the tarball contains the XUL.framework directory directly (rather than xulrunner/XUL.framework).

Comment 8

[Benjamin Smedberg]

Comment on attachment 612399 [details] [diff] [review]
bustage fix

Yeah, I got halfway through writing the same patch and got stuck! thanks for finishing this.

Comment 9

[Dave Townsend [:mossop]]

Landed on inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/d36aeec1abe1

Comment 10

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/d36aeec1abe1

Comment 11

[Alex Vincent [:WeirdAl]]

This is a pretty significant change for XULRunner app developers.  The XULRunner docs could use some tender loving care, anyway...