Closed Bug 1254281 Opened 5 years ago Closed 4 years ago

Build system changes for the transitioned Gaia


(Firefox OS Graveyard :: Gaia::Build, defect)

Gonk (Firefox OS)
Not set


(Not tracked)



(Reporter: fabrice, Assigned: gerard-majax)




(3 files)

This is to prepare for a chrome:// served Gaia. Instead of packaging each app independently in an file, we will package them all together, and put the shared/ alongside the apps.
The directory tree will look like:

|- shared
|- system
|- settings
|- ...

Compared to today's builds, we have to:
- stop including shared/ in every app.
- still select images at the appropriate resolution.
- stop packaging apps individually.
- stop using the subdomain.
I'll try to hack on this.
Assignee: nobody → lissyx+mozillians
Current status is that building profile fails around settings + shared with require.js doing stuff.
(In reply to Alexandre LISSY :gerard-majax from comment #3)
> Current status is that building profile fails around settings + shared with
> require.js doing stuff.

Should be fixed now, builds profile/webapps and hacked to copy shared/:
> LC_ALL=C ll profile/webapps/
> total 32K
> drwxr-xr-x  7 alex alex 4.0K Mar 14 14:20 .
> drwxrwxr-x  5 alex alex 4.0K Mar 14 14:20 ..
> drwxr-xr-x  7 alex alex 4.0K Mar 14 14:20 homescreen
> drwxr-xr-x  6 alex alex 4.0K Mar 14 14:20 keyboard
> drwxr-xr-x  8 alex alex 4.0K Mar 14 14:20 settings
> drwxr-xr-x  9 alex alex 4.0K Mar 14 14:20 shared
> drwxr-xr-x 11 alex alex 4.0K Mar 14 14:20 system
> -rw-rw-r--  1 alex alex 1.4K Mar 14 14:20 webapps.json
With bug 1256506 moving forward, I am going to try and wrap up this to push to kanikani by the end of the week :)
Good news: after some more path hacking, I am getting into a state where:
 - make clean && make PRODUCTION=1 profile
 - adb shell rm -r /system/b2g/apps/*
 - adb push profile/webapps/ /system/b2g/apps/

Results in system app basically booting, with few enough errors so that homescreen background is visible, lockscreen is visible (with date, time and slider working) and opening a new browser window works.
Depends on: 832165
Used to identify the places where we want to change "webapps" to "apps" to transition from /system/b2g/webapps to /system/b2g/apps

Aus, this might likely break test infras :)
Flags: needinfo?(aus)
make APP=xxx install-gaia fixed too.
make reset-gaia works
and we are now pushing/installing to /system/b2g/apps, I am fixing |./| to be working properly with this and it will be good to merge.
This is suboptimal, but Android build system really don't seem to love leaving out of his build tree. So you might have to deal with changing your B2G build tree's gaia/ to reference "apps" instead of "webapps".
I don't think we need/can land those outside of pine/kanikani, too intrusive. On the other hand, it's just a one time easy to apply patch.
Time to land, this is booting, we can iterate over from this.
Closed: 4 years ago
Resolution: --- → FIXED
Blocks: 1257925
Depends on: 1258345
FYI, turns out building with GAIA_PATH exported does not really work that well. Please refrain from trying to do this and just checkout gaia kanikani in your B2G/gaia/.
Thanks for the heads up, now that the tests are running as expected again, I can go ahead and fix those. =)
Flags: needinfo?(aus)
You need to log in before you can comment on or make changes to this bug.