Closed Bug 1186008 Opened 9 years ago Closed 9 years ago

[Shinano] Fix devices naming

Categories

(Firefox OS Graveyard :: GonkIntegration, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
FxOS-S4 (07Aug)

People

(Reporter: gerard-majax, Assigned: gerard-majax)

References

Details

Attachments

(4 files)

We should stop naming the device 'shinano' because that is the name of the platform. We are using this name on lollipop also which makes the Z3 device not buildable because all the Sony bits expects 'leo' and not 'shinano'. The plan is to: - kill the old 'leo' (LG Fireweb) device since it's deprecated and not used anymore - update docs (MDN/intranet) - rename 'shinano'/'shinano-l' device (PRODUCT_NAME) to 'leo-kk' and 'leo-l' depending on the port - rename 'aries'/'aries-l' device (PRODUCT_NAME) to 'aries-kk' and 'aries-l' I think we should keep the 'shinano', 'shinano-l' and 'aries' entries in config.sh for backward compability, but make sure we set the proper PRODUCT_NAME.
What will be the impact for currently used devices ? Those will be impacted by the renaming of 'aries' to 'aries-kk'.
Flags: needinfo?(wcosta)
Flags: needinfo?(nhirata.bugzilla)
Flags: needinfo?(drs)
Flags: needinfo?(bhearsum)
Deferring to Wander, he's doing some work in this area already.
Flags: needinfo?(bhearsum)
I would prefer not to change at all. Esp because I'm already busy with things + I need to figure out when I can test things. If we are to change: 1) Not too sure if contributors are building for Leo ( LG Fireweb )since they couldn't figure out how to root it. 2) I would prefer to rename LG leo rather than delete it. If we're going to seriously delete it then we should delete all the other ones like Otoro, Unagi, etc. 3) At the same time, I think it might be best to name it something like LG-Leo and Sony-Leo; I have a feeling that there's going to be multiple Leos. Aries-kk, pvtbuild, task cluster + the update server has to end up changing as well as automation. The getting people (QA + dogfooders) to be aware of this, and the new location. The cost of this change will cause a lot of breakage all around, and confusion. This is based on the size change for flame.
Flags: needinfo?(nhirata.bugzilla)
Note: the system size for flame was needed for OTA. This change is not necessarily needed. It was a bad choice of names and if we're already keeping the shinano for backwards compatibility, why are we changing it now?
Flags: needinfo?(lissyx+mozillians)
(In reply to Naoki Hirata :nhirata (please use needinfo instead of cc) from comment #4) > Note: the system size for flame was needed for OTA. > > This change is not necessarily needed. It was a bad choice of names and if > we're already keeping the shinano for backwards compatibility, why are we > changing it now? When I'm speaking of compatibility, I'm just referring to the config.sh argument. (In reply to Naoki Hirata :nhirata (please use needinfo instead of cc) from comment #3) > I would prefer not to change at all. Esp because I'm already busy with > things + I need to figure out when I can test things. > > If we are to change: > 1) Not too sure if contributors are building for Leo ( LG Fireweb )since > they couldn't figure out how to root it. > 2) I would prefer to rename LG leo rather than delete it. If we're going to > seriously delete it then we should delete all the other ones like Otoro, > Unagi, etc. > 3) At the same time, I think it might be best to name it something like > LG-Leo and Sony-Leo; I have a feeling that there's going to be multiple Leos. > > Aries-kk, pvtbuild, task cluster + the update server has to end up changing > as well as automation. The getting people (QA + dogfooders) to be aware of > this, and the new location. The cost of this change will cause a lot of > breakage all around, and confusion. This is based on the size change for > flame. Yeah that was the kind of thing I was fearing about. I have just re-read the email reply from Michael about this, and I don't know where the idea to rename aries came from, but it's not there. So I'll just hack on Z3, but do not touch Z3 Compact.
Flags: needinfo?(wcosta)
Flags: needinfo?(lissyx+mozillians)
Flags: needinfo?(drs)
Should we won't fix this then? Or do you have a different proposal?
Flags: needinfo?(lissyx+mozillians)
No, we really need to fix the Z3.
Flags: needinfo?(lissyx+mozillians)
Attached file Manifest PR (deleted) —
Michael, this does not include any change on the naming of KK repo and device directory for Z3, which I'm sure we do want. So for KK, should I: - just rename the device/sony/shinano to device/sony/leo and keep the repo name as device-shinano on the mozilla-b2g github account - or should we also rename the device-shinano to device-leo (and thus delete the LG device) ?
Attachment #8637076 - Flags: review?(mwu)
Attached file B2G PR (deleted) —
Removing references to the old leo LG device, renaming shinano to leo-kk and shinano-l to leo-l.
Attachment #8637079 - Flags: review?(mwu)
Attached file Sony Leo device PR (deleted) —
Renaming the L device
Attachment #8637081 - Flags: review?(mwu)
Attached file Sony Leo device KK PR (deleted) —
And now the KK device counterpart.
Attachment #8637083 - Flags: review?(mwu)
Kevin, I know you have such a device so you might want to be aware. Would be awesome if you have some time to cross check on your side that everything is good :).
Flags: needinfo?(kgrandon)
No longer blocks: 1176287
(In reply to Alexandre LISSY :gerard-majax from comment #8) > Created attachment 8637076 [details] > Manifest PR > > Michael, this does not include any change on the naming of KK repo and > device directory for Z3, which I'm sure we do want. > > So for KK, should I: > - just rename the device/sony/shinano to device/sony/leo and keep the repo > name as device-shinano on the mozilla-b2g github account > - or should we also rename the device-shinano to device-leo (and thus > delete the LG device) ? Isn't there a device-sony-leo we can use? Just keeping the repo name as device-shinano is also fine. I don't care too much about the details here if we're going to be focusing on the l port.
The device-sony-leo is from the fork from Sony's repo. While we could push to a dedicated branch, I think it may lead to more confusion than anything else.
Flags: needinfo?(mwu)
Ok
Flags: needinfo?(mwu)
Attachment #8637076 - Flags: review?(mwu) → review+
Attachment #8637079 - Flags: review?(mwu) → review+
Attachment #8637081 - Flags: review?(mwu) → review+
Attachment #8637083 - Flags: review?(mwu) → review+
FYI wander, there might be some fall outs from this? Not sure. Hopefully the new name will not cause issues.... Peter, Kevin, please keep an eye out to make sure we have working aries build tomorrow.
Flags: needinfo?(wcosta)
Flags: needinfo?(pbylenga)
Flags: needinfo?(ktucker)
Sorry for the late response. aries.xml is a sym link to shinano.xml in the b2g-manifest repo. As shinano is still there, we should be fine. I am working on unique device names for OTA things, but it is unrelated to this.
Flags: needinfo?(wcosta)
(In reply to Alexandre LISSY :gerard-majax from comment #12) > Kevin, I know you have such a device so you might want to be aware. Would be > awesome if you have some time to cross check on your side that everything is > good :). I can try. So I should re-build, with 'leo-l'?
Flags: needinfo?(kgrandon)
(In reply to Kevin Grandon :kgrandon from comment #19) > (In reply to Alexandre LISSY :gerard-majax from comment #12) > > Kevin, I know you have such a device so you might want to be aware. Would be > > awesome if you have some time to cross check on your side that everything is > > good :). > > I can try. So I should re-build, with 'leo-l'? Kevin, I think you probably have a KK build right now. Check your .config in your build tree. If you have "shinano", then pull to update your B2G/ tree and then |./config.sh leo-kk|. If you have "shinano-l", you'll need to |./config.sh leo-l|.
Flags: needinfo?(kgrandon)
Still on my radar to test, I think I need the appropriate backup directory. Will follow-up on that.
Flags: needinfo?(ktucker)
clearing NI, didn't see anything abnormal on Aries
Flags: needinfo?(pbylenga)
Target Milestone: --- → FxOS-S4 (07Aug)
(In reply to Alexandre LISSY :gerard-majax from comment #12) > Kevin, I know you have such a device so you might want to be aware. Would be > awesome if you have some time to cross check on your side that everything is > good :). Been dogfooding for today with this (leo-kk build) so far and nothing that seems off. Will report back if I find anything. Thanks for doing this work!
Flags: needinfo?(kevingrandon)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: