Closed
Bug 1130159
Opened 9 years ago
Closed 9 years ago
Disable installing apns-conf.xml by default
Categories
(Firefox OS Graveyard :: GonkIntegration, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
2.2 S6 (20feb)
People
(Reporter: mwu, Assigned: mwu)
Details
Attachments
(2 files)
I don't think we use apns-conf.xml in our radio code, and many manifests don't have the goldfish repo, so let's remove this.
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8560090 -
Flags: review?(htsai)
Comment 2•9 years ago
|
||
Comment on attachment 8560090 [details] [review] Don't install the goldfish apns-conf.xml by default Correct, our ril code doesn't use this. Thank you.
Attachment #8560090 -
Flags: review?(htsai) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 3•9 years ago
|
||
https://github.com/mozilla-b2g/platform_build/commit/e9c68a8f768d569622fe09d05a5be59d0bd86baa
Status: NEW → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S6 (20feb)
Hello, I know that JB is no longer maintained, but is it possible to make an exception ? It's necessary for building a full JB image for Open C.
Attachment #8715289 -
Flags: review?(htsai)
Comment 5•8 years ago
|
||
(In reply to micgeri from comment #4) > Created attachment 8715289 [details] [review] > Cherry-pick "Don't install the goldfish apns-conf.xml by default" patch for > JB > > Hello, > > I know that JB is no longer maintained, but is it possible to make an > exception ? It's necessary for building a full JB image for Open C. Hi Sorry for the delay. It was Chinese new year holiday the last week. I think cherry picking this for JB makes no harm so it's fine to me. But I'd like to hear Michael's opinion. Thanks!
Flags: needinfo?(mwu)
Updated•8 years ago
|
Attachment #8715289 -
Flags: review?(htsai)
(In reply to Hsin-Yi Tsai [:hsinyi] from comment #5) > (In reply to micgeri from comment #4) > > Created attachment 8715289 [details] [review] > > Cherry-pick "Don't install the goldfish apns-conf.xml by default" patch for > > JB > > > > Hello, > > > > I know that JB is no longer maintained, but is it possible to make an > > exception ? It's necessary for building a full JB image for Open C. > > Hi Sorry for the delay. It was Chinese new year holiday the last week. > > I think cherry picking this for JB makes no harm so it's fine to me. But I'd > like to hear Michael's opinion. Thanks! Hello, Ok, no problem
Assignee | ||
Comment 7•8 years ago
|
||
merged https://github.com/mozilla-b2g/platform_build/commit/8df1b0580ade1786bc1bf05a6abc06bcd62c2d6b
Flags: needinfo?(mwu)
You need to log in
before you can comment on or make changes to this bug.
Description
•