Closed Bug 1133347 Opened 5 years ago Closed 5 years ago

Migrate the classic homescreen into its own project

Categories

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

x86
macOS
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
2.2 S11 (1may)

People

(Reporter: kgrandon, Assigned: kgrandon)

References

Details

(Whiteboard: [systemsfe])

Attachments

(2 files)

We should migrate the classic home screen into a new github repo and upload it to the marketplace.
Blocks: 1134952
Blocks: 1135334
Blocks: 1139374
(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?
(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.
Maybe just moving it to a $gaia/attic would be good enough?
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.
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?
Flags: needinfo?(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
Flags: needinfo?(crdlc)
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 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.
Attachment #8564755 - Flags: review?(timdream)
Attachment #8564755 - Flags: review?(ricky060709)
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.
Attachment #8564755 - Flags: review?(timdream)
(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 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!
Attachment #8564755 - Flags: review?(crdlc)
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.
Attachment #8564755 - Flags: review?(ricky060709) → review+
Comment on attachment 8564755 [details] [review]
[gaia] KevinGrandon:bug_1133347_remove_classic_homescreen > mozilla-b2g:master

thanks a lot
Attachment #8564755 - Flags: review?(crdlc) → review+
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.
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Landed a follow-up to remove old test definitions from tbpl-manifest: https://github.com/mozilla-b2g/gaia/commit/eff65532a7b5a4edca995cf05b8f3ec5b56aaafa
Whiteboard: [systemsfe]
Target Milestone: --- → 2.2 S11 (1may)
You need to log in before you can comment on or make changes to this bug.