Closed Bug 672897 Opened 10 years ago Closed 10 years ago

Update compatibility info for Feedback add-on automatically

Categories

(Firefox for Android Graveyard :: Extension Compatibility, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 8

People

(Reporter: mbrubeck, Assigned: mbrubeck)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch patch (obsolete) — Splinter Review
+++ This bug was initially created as a clone of Bug #659468 +++

This copies the Makefile code from bug 659468 to preprocess the add-on install.rdf, so we won't need to bump it manually every six weeks.
Attachment #547201 - Flags: review?(mark.finkle)
Comment on attachment 547201 [details] [diff] [review]
patch

Have you tested this on Android? I don't think we currently unpack the "distribution/extensions" folder from the APK. We do unpack "extensions" folder. We'd need to add "distribution/extensions"
Attached patch patchSplinter Review
Updated to install to dist/bin/extensions, and tested successfully on Android.
Attachment #547201 - Attachment is obsolete: true
Attachment #547206 - Flags: review?(mark.finkle)
Attachment #547201 - Flags: review?(mark.finkle)
Comment on attachment 547206 [details] [diff] [review]
patch

Someday we should change to "distribution/extensions" as that location allows for updates to add-ons from AMO. Those add-ons in "extensions" can't be updated via AMO.

Different bug...
Attachment #547206 - Flags: review?(mark.finkle) → review+
http://hg.mozilla.org/mozilla-central/rev/2b4cf59fa42c
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 8
Whiteboard: [inbound]
Verified fixed on:
Mozilla/5.0 (Android;Linux armv7l;rv:8.0)Gecko/20110928
Firefox/8.0 Fennec/8.0
Device: Samsung Galaxy S
OS: Android 2.2
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.