Closed Bug 1130159 Opened 9 years ago Closed 9 years ago

Disable installing apns-conf.xml by default

Categories

(Firefox OS Graveyard :: GonkIntegration, defect)

All
Gonk (Firefox OS)
defect
Not set
normal

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.
Attachment #8560090 - Flags: review?(htsai)
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+
Keywords: checkin-needed
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)
(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)
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
Thanks
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: