Closed Bug 679575 Opened 8 years ago Closed 8 years ago

Aegis should be enabled for Maemo6 define (not only for content manager)

Categories

(Firefox for Android Graveyard :: General, defect)

x86
Linux
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 9

People

(Reporter: romaxa, Assigned: romaxa)

Details

Attachments

(1 file, 2 obsolete files)

Aegis permissions required not only for tracker, but also for geolocation.. so aegis files should be enabled for maemo6 define
Assignee: nobody → romaxa
Status: NEW → ASSIGNED
Attachment #553644 - Flags: review?(doug.turner)
this is needed to turn-on all aegis permissions for fennec, most important one is Geolocation provider
Uh, found that filter substitution was not added to current aegis file... fixing it also here
Attachment #553644 - Attachment is obsolete: true
Attachment #553644 - Flags: review?(doug.turner)
Attachment #554014 - Flags: review?(doug.turner)
Attachment #554014 - Flags: review?(doug.turner) → review+
Keywords: checkin-needed
More bitrot, or so it seems...

applying 679575
patching file mobile/installer/Makefile.in
Hunk #1 FAILED at 173
1 out of 2 hunks FAILED -- saving rejects to file mobile/installer/Makefile.in.rej
patching file mobile/installer/debian/fennec.aegis.in
Hunk #1 FAILED at 0
1 out of 1 hunks FAILED -- saving rejects to file mobile/installer/debian/fennec.aegis.in.rej
patch failed, unable to continue (try -v)
patch failed, rejects left in working dir
errors during apply, please fix and refresh 679575
Keywords: checkin-needed
Attachment #554014 - Attachment is obsolete: true
Attachment #554014 - Attachment is obsolete: false
Attachment #554443 - Attachment description: Enable aegis for maemo6 (not only for content manager). refreshed → [TUPUSH] Enable aegis for maemo6 (not only for content manager). refreshed
Keywords: checkin-needed
Attachment #554014 - Attachment is obsolete: true
Had rotted again due to your patch in bug 679987 landing, but have just fixed it locally to save another back and forth.

Sent to try:
http://tbpl.allizom.org/?tree=Try&usebuildbot=1&rev=9a567d609584

Assuming all green, will push to inbound after :-)
Keywords: checkin-needed
Target Milestone: --- → Firefox 9
http://hg.mozilla.org/mozilla-central/rev/30e7c0dfeb3a
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.