Closed Bug 1211870 Opened 9 years ago Closed 7 years ago

A manifest to use CyanogenMod plus an optional alternative manifest for sony-aosp-l devces

Categories

(Firefox OS Graveyard :: GonkIntegration, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: afarden, Assigned: afarden)

References

Details

Attachments

(2 files)

52 bytes, text/x-github-pull-request
Details | Review
43 bytes, text/x-github-pull-request
Details | Review
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/45.0.2454.101 Safari/537.36

Steps to reproduce:

CyanogenMod's build repo automatically builds a device's kernel and selects an appropriate audio/display/media HAL (others too) so If we want to support CynogenMod repos we should use this too.
Current progress: swapping only the build repo, adjusting Sony AOSP repos with necessary changes to ensure all is building correctly.
Blocks: 1080483
Summary: Use CyanogenMod's build repo for inline kernel building and HAL switching → Use CyanogenMod repos to build sony-aosp-l devces
Currently we're not yet using vendor/cm so you need to force the CM_VERSION, just add CM_VERSION := 12.1 to your device repo
Attached file Manifest PR
This manifest references repos outside of github.com/mozilla-b2g, in github.com/cm-b2g.

This is not ideal, but given the circumstances - the porting hub is launching in a matter of hours and there isn't even a useful porting guide - we need something that works NOW.

There is of course the option of making github.com/cm-b2g the official location for all cm based ports, I'm more than happy to add ownership to the relevant people at Mozilla. This will also keep repos with any potential licencing issues away from Mozilla repos.

If that is not desirable there are a total of seven essential external repos, most of which already have Mozilla forks and can be brought into existing mozilla-b2g repos.
Attachment #8681688 - Flags: review?(lissyx+mozillians)
Attachment #8681688 - Flags: feedback?(mwu)
Attached file B2G PR
instead of a symlink to a device specific manifest, we use a symlink to the base-l-cm manifest. The guide will instruct porters to add their device specific repos to a local manifest.

This will simplify and speed up porting, as there is no need to fork and create manifests and edit config files which the new porter is not likely to immediately understand.

The porters can create a device specific manifest after the port is booting.
Attachment #8681692 - Flags: review?(lissyx+mozillians)
Attachment #8681692 - Flags: feedback?(mwu)
I'd prefer we keep those repos on mozilla-b2g account :)
Summary: Use CyanogenMod repos to build sony-aosp-l devces → A manifest to use CyanogenMod plus an optional alternative manifest for sony-aosp-l devces
(In reply to Alexandre LISSY :gerard-majax from comment #6)
> I'd prefer we keep those repos on mozilla-b2g account :)

Michael, what is your opinion on that?
Flags: needinfo?(mwu)
Assignee: nobody → afarden
If we control it, github.com/mozilla-b2g makes more sense to me. I don't think we get to just ignore licensing issues.
Flags: needinfo?(mwu)
Before we fork anything Peter wants to clear it with legal. I've already sent him details, that forking CM frameworks is the same as forking AOSP frameworks - it's all 100% open source - and that no device specific code is included.

Hopefully we can clear that hurdle soon and worry about the device specific parts later. Just simply having the frameworks and manifest available will help us immensely.
What is the status regarding comment 9 ?
Flags: needinfo?(pdolanjski)
Flags: needinfo?(ellee)
Unfortunately no updates quite yet. I added the legal bug as a blocker though.
Flags: needinfo?(ellee)
Flags: needinfo?(pdolanjski)
Status: UNCONFIRMED → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Attachment #8681688 - Flags: review?(lissyx+mozillians)
Attachment #8681688 - Flags: feedback?(mwu.code)
Attachment #8681692 - Flags: review?(lissyx+mozillians)
Attachment #8681692 - Flags: feedback?(mwu.code)
You need to log in before you can comment on or make changes to this bug.