1133347 - Migrate the classic homescreen into its own project

Status: RESOLVED FIXED

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

Description

[Kevin Grandon :kgrandon]

We should migrate the classic home screen into a new github repo and upload it to the marketplace.

Comment 1

[GitHub Autolander Bot]

Attachment: [gaia] KevinGrandon:bug_1133347_remove_classic_homescreen > mozilla-b2g:master

Comment 2

[[:fabrice] Fabrice Desré]

(In reply to Kevin Grandon :kgrandon from comment #0)
> We should migrate the classic home screen into a new github repo and upload
> it to the marketplace.

Is it using only privileged apis?

Comment 3

[Kevin Grandon :kgrandon]

(In reply to Fabrice Desré [:fabrice] from comment #2)
> Is it using only privileged apis?

Not yet, I guess that's something we would need to fix before uploading. Also would need to figure out the gaia shared/build pieces as well.

Comment 4

[[:fabrice] Fabrice Desré]

Maybe just moving it to a $gaia/attic would be good enough?

Comment 5

[Kevin Grandon :kgrandon]

I do think it would be very compelling to upload this to the Marketplace. It would be a good use of the replaceable home screen feature. Archiving it in attic/ would probably be fine initially though.

Comment 6

[Kevin Grandon :kgrandon]

Cristian - what are your thoughts on uploading the classic home screen into its own repo under the mozilla-b2g org? I'd like to see about spinning this off and potentially using it for a replaceable home screen demo. Is this work that you'd be interested in doing and want to help with?

Comment 7

[Cristian Rodriguez (:crdlc)]

(In reply to Kevin Grandon :kgrandon from comment #6)
> Cristian - what are your thoughts on uploading the classic home screen into
> its own repo under the mozilla-b2g org? I'd like to see about spinning this
> off and potentially using it for a replaceable home screen demo. Is this
> work that you'd be interested in doing and want to help with?

Hi Kevin, I don't have problem with this idea. Quite the opposite, I would be nice. But I cannot take this because I have other tasks in my company right now and I cannot work on this. Sorry for that

Comment 8

[Kevin Grandon :kgrandon]

I've done an initial import of the classic home screen into this repository: https://github.com/mozilla-b2g/classic-homescreen

I'm going to spend some time getting it to work as a privileged home screen next.

Comment 9

[Kevin Grandon :kgrandon]

Comment on attachment 8564755 [details] [review]
[gaia] KevinGrandon:bug_1133347_remove_classic_homescreen > mozilla-b2g:master

Ricky, Tim - please be careful loading this, the diff is quite large and causes my browser some problems.

Ricky - if you could please review the changes to the build system, which should currently be contained within this commit: https://github.com/KevinGrandon/gaia/commit/a696ba8b0d71a5b521b13ebd580491fad1ba0ea2

Tim - just looking for a sign off if this looks good to you. The home screen has been migrated to this external repository: https://github.com/mozilla-b2g/classic-homescreen There's still a bit of work to do here including figuring out what's needed for a build system (might be good for a modular build system) as well as localization efforts if we desire.

Comment 10

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 8564755 [details] [review]
[gaia] KevinGrandon:bug_1133347_remove_classic_homescreen > mozilla-b2g:master

I don't need to sign off the changes -- you should ask one of the Gaia:Homescreen owners. I however would advice we hold off this fix until we could either get a firm confirmation from the product side on not shipping the app EVER, or we could make sure classic-homesceen repo can be integrated w/o a lot of troubles. 

Also in the classic-homesceen repo, I would use git filter-branch to preserve the history of home screen app, but that's up to you.

Comment 11

[Kevin Grandon :kgrandon]

(In reply to Tim Guan-tin Chien [:timdream] (slow response; please ni? to queue) from comment #10)
> Comment on attachment 8564755 [details] [review]
> [gaia] KevinGrandon:bug_1133347_remove_classic_homescreen >
> mozilla-b2g:master
> 
> I don't need to sign off the changes -- you should ask one of the
> Gaia:Homescreen owners. I however would advice we hold off this fix until we
> could either get a firm confirmation from the product side on not shipping
> the app EVER, or we could make sure classic-homesceen repo can be integrated
> w/o a lot of troubles. 

Sounds good. The plan is to move this into an external repo, and if we need to re-integrate it we have a few options. We can re-land, but my recommendation would be to do what we do with loop - using the outoftree_apps folder.

> Also in the classic-homesceen repo, I would use git filter-branch to
> preserve the history of home screen app, but that's up to you.

Sounds good, I will look at doing this.

Comment 12

[Kevin Grandon :kgrandon]

Comment on attachment 8564755 [details] [review]
[gaia] KevinGrandon:bug_1133347_remove_classic_homescreen > mozilla-b2g:master

Cristian - could you sign off on this? The work to migrate to an external repo is in progress, and the app should be working there soon. The goal is to get this uploaded to the marketplace. Thanks!

Comment 13

[Ricky Chien [:rickychien] (inactive)]

Comment on attachment 8564755 [details] [review]
[gaia] KevinGrandon:bug_1133347_remove_classic_homescreen > mozilla-b2g:master

I don't see any potential problem in this patch, r+ for build part.

Comment 14

[Cristian Rodriguez (:crdlc)]

Comment on attachment 8564755 [details] [review]
[gaia] KevinGrandon:bug_1133347_remove_classic_homescreen > mozilla-b2g:master

thanks a lot

Comment 15

[Kevin Grandon :kgrandon]

Awesome, I was able to preserve history and did a new import: https://github.com/mozilla-b2g/classic-homescreen/commits/1961bb0f9e9f08471c9c6222cf816044b3385279

Adding checkin-needed as we are good to go here and can start working on the external repository. There's some gecko bugs to fix before it will fully work as a privileged app.

Comment 16

[GitHub Autolander Bot]

Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/f2031882e90a21e5274a09e8f9805a1b2ce5403c

Comment 17

[GitHub Autolander Bot]

Attachment: [gaia] KevinGrandon:bug_1133347_remove_old_legacy_definitions > mozilla-b2g:master

Comment 18

[Kevin Grandon :kgrandon]

Landed a follow-up to remove old test definitions from tbpl-manifest: https://github.com/mozilla-b2g/gaia/commit/eff65532a7b5a4edca995cf05b8f3ec5b56aaafa