1010014 - [Build System] homescreen.manifestURL should be customizable.

Status: RESOLVED FIXED

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

Description

[GaryChen [:GaryChen][:PYChen][:gchen][:陳柏宇]]

We have 3 homescreen APPs in GAIA code base and homescreen APP may expose to privileged in the next phase.
We need a way to customize 'default' homescreen APP for first launch.

Comment 1

[GaryChen [:GaryChen][:PYChen][:gchen][:陳柏宇]]

Attachment: pull request: https://github.com/mozilla-b2g/gaia/pull/19227

Hi Yuren,
   I create a pseudo key -- 'homescreen.appName' as a customization entry key.
   The function |setHomescreenURL| for composing |homescreen.manifestURL|.

   Please give me some feedback with this patch or welcome to discuss in person.
   
   I will add more test case after we confirm this patch is on the right way.
   Thanks.

Comment 2

[Yuren [:yurenju] (AFK ~ Oct. 12) (volunteer mode)]

Comment on attachment 8422260 [details] [review]
pull request: https://github.com/mozilla-b2g/gaia/pull/19227

overall looks good, but please raise an exception if the homescreen app isn't in GAIA_APPDIRS.

Comment 3

[GaryChen [:GaryChen][:PYChen][:gchen][:陳柏宇]]

Comment on attachment 8422260 [details] [review]
pull request: https://github.com/mozilla-b2g/gaia/pull/19227

Hi Yuren,
   I found the redundant variable in https://github.com/mozilla-b2g/gaia/pull/19227/files#diff-b67911656ef5d18c4ae36cb6741b7965L465.
   So I remove it and I add more test in this patch.
   Please help to review this patch again.
   
   Thanks.

Comment 4

[Yuren [:yurenju] (AFK ~ Oct. 12) (volunteer mode)]

Comment on attachment 8422260 [details] [review]
pull request: https://github.com/mozilla-b2g/gaia/pull/19227

r=yurenju if issues on github are addressed.

Comment 5

[GaryChen [:GaryChen][:PYChen][:gchen][:陳柏宇]]

landed in master:
https://github.com/mpizza/gaia/commit/6b2c47438fb02c76310f07600d4d7e5d808f482e

travis got green:
https://travis-ci.org/mozilla-b2g/gaia/builds/25774171