791267 - We can now use CPU architecture (ABI) for multiple APK filtering

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

In bug 788666, we needed to hack ARMv7 and ARMv6 multiple APKs to use screensize as the way to make the configurations be different. Google now supports "CPU architecture (ABI)" as a way to make the configurations be different.

This should mean we can backout the hack added in bug 788666.

http://developer.android.com/guide/google/play/publishing/multiple-apks.html

Comment 1

[Alex Keybl [:akeybl]]

What timing.. at least we can remove the hack though!

Comment 2

[Alex Keybl [:akeybl]]

We'll want to test on a Google Play test product before officially making this change.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

There is a test build here:
https://play.google.com/store/apps/details?id=org.mozilla.fennec_blassey

The build has a backout of bug 788666. Can we get some QA testing to see if ARMv6 and ARMv7 works ok on that build?

Comment 4

[Aaron Train [:aaronmt]]

Works. Tried the HTC Wildfire S, HTC Legend, LG Slider and each got an ~18MB ARMv6 build. Galaxy Nexus/Nexus 7 both got an ~17.3 MB ARMv7 build.

Comment 5

[Brad Lassey [:blassey] (use needinfo?)]

Note that finkle just took down the ARMv6 build in the market. Any further testing will be broken until further notice.

Comment 6

[Brad Lassey [:blassey] (use needinfo?)]

This has been fixed, resume testing

Comment 7

[Matt Brubeck (:mbrubeck)]

Marking this as fixed for Fx17 based on comment 4.  This also means we can back out the patch from bug 788666 on trunk; leaving the bug open until that is done.

Comment 8

[Matt Brubeck (:mbrubeck)]

Attachment: back out bug 788666

Revert this manifest change that we added to work around Google Play limitations.

Comment 9

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/eeaec0df48ee

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/eeaec0df48ee