816687 - Add option for forcing gaia apps to install to /system

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Comment 1

[Diego Wilson [:diego]]

Attachment: Gaia force system apps

Comment 2

[Diego Wilson [:diego]]

Attachment: Gonk-misc force system apps

Comment 3

[Diego Wilson [:diego]]

The gaia and gonk-misc patches above add an option to force installing gaia apps to /system by setting B2G_SYSTEM_APPS=1

Comment 4

[[:fabrice] Fabrice Desré]

How is this different from building with VARIANT=user ?

Comment 5

[Diego Wilson [:diego]]

Excellent question. As you may already know, -user and -userdebug differ from -eng builds in that:

* -eng builds have a larger set of apps installed including several test apps
* -userdebug builds require "adb root" to be called explicitly

Among other things. Many devs prefer to use -eng for these reasons. However, some reference devices do not have a lot of /data space to spare. The patches above allow devs to build a -eng build that fits in these devices.

Comment 6

[Michael Wu [:mwu]]

Comment on attachment 686762 [details] [diff] [review]
Gonk-misc force system apps

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

::: Android.mk
@@ +146,2 @@
>  PRESERVE_B2G_WEBAPPS := 1
>  B2G_UPDATER ?= 1

Was moving B2G_UPDATER here intentional? It doesn't seem to me that installing apps to /system should imply the updater should be enabled.

Comment 7

[Diego Wilson [:diego]]

Attachment: Gonk-misc force system apps

Comment 8

[Diego Wilson [:diego]]

(In reply to Michael Wu [:mwu] from comment #6)
> Comment on attachment 686762 [details] [diff] [review]
> Gonk-misc force system apps
> 
> Review of attachment 686762 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: Android.mk
> @@ +146,2 @@
> >  PRESERVE_B2G_WEBAPPS := 1
> >  B2G_UPDATER ?= 1
> 
> Was moving B2G_UPDATER here intentional? It doesn't seem to me that
> installing apps to /system should imply the updater should be enabled.

Moved back up

Comment 9

[Diego Wilson [:diego]]

https://github.com/mozilla-b2g/gaia/pull/6809
https://github.com/mozilla-b2g/gonk-misc/pull/59

Comment 10

[amiller]

Device: Unagi
Build: 20130112070202
Does not reproduce at this time. Gonk updated