Closed Bug 1231834 Opened 8 years ago Closed 8 years ago

Add manifest file for shared B2G repositories

Categories

(Firefox OS Graveyard :: General, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: tzimmermann, Assigned: tzimmermann)

Details

Attachments

(1 file)

Some repositories are shared by all targets. We should put them in a separate file for convenience.
Attached file Github pull request
This is supposed to simplify repo handling a bit.
Attachment #8697417 - Flags: review?(jgomez)
I disagree with moving all remotes to a single file. Common remotes might fine, but device specific ones really do not belong.
(In reply to Michael Wu [:mwu] from comment #2)
> I disagree with moving all remotes to a single file. Common remotes might
> fine, but device specific ones really do not belong.

Yeah. I was undecided about this. There are some remotes that are used by more than one manifest, but only a few; like "linaro". Do you have a suggestion where to draw the line for sharing remotes? More than one or two targets?
I think that by common remotes, we mean: aosp, caf, mozilla and b2g. All other remotes are usually very device specific (sprd, t2m, allwinner, ... ). So I'd suggest to move the former to a common file and keep the latter into the specific device manifest.
Comment on attachment 8697417 [details] [review]
Github pull request

Updated Github pull request

  * moved device-specific repos back into device manifests
  * apitrace it used by almost every device, so I kept it in base.xml
Attachment #8697417 - Flags: review?(jgomez) → review+
Comment on attachment 8697417 [details] [review]
Github pull request

Changes to Github pull request

  - rebased onto latest master branch
Apparently no build problems showed up. I'm closing the bug as FIXED.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: