1015949 - mulet's package doesn't contain b2g specifics

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Runtime, defect)

Description

[Alexandre Poirot [:ochameau]]

When building `package` target on mulet, you get a Firefox tarball, without any b2g specific being explicitely listed in:
  http://mxr.mozilla.org/mozilla-central/source/b2g/installer/package-manifest.in

We need to ship those files in mulet packages!

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: patch

https://tbpl.mozilla.org/?tree=Try&rev=80c20c7d5ca9

This patch removed the b2g.js pref file hack from browser/
Instead we now ensure preprocessing, building and shipping this file from b2g/dev/app/ and b2g/installer/.

And I added a reference to b2g package-manifest from browser's Makefile.
I had to toggle MOZ_PKG_FATAL_WARNING to 0 as b2g manifest has many duplicated files with Firefox one.

Comment 2

[[:fabrice] Fabrice Desré]

Comment on attachment 8428771 [details] [diff] [review]
patch

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

lgtm, but I would not mind a second pair of eyes here.

Comment 3

[Michael Shal [:mshal]]

Comment on attachment 8428771 [details] [diff] [review]
patch

Looks good to me too.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/b2g-inbound/rev/8331da4219d8

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/8331da4219d8

Comment 6

[Alexandre Poirot [:ochameau]]

Attachment: land missing b2g/dev/app/moz.build file.

I forgot to git add this file before submitting the patch...

Comment 7

[Alexandre Poirot [:ochameau]]

I don't have try run as attachment 8431821 [details] [diff] [review] is mulet specific and mulet isn't built yet.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/fx-team/rev/c387f539ede2

Comment 9

[Phil Ringnalda (:philor)]

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