602846 - no feedback when extension isn't compatible due to em:targetPlatform

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[(not currently active) Ted Mielczarek]

I was trying to install my "Crash me now!" extension on Fennec on Android. It has a binary component, so it uses em:targetPlatform to be specific. I was missing one for Android_arm-eabi-gcc3, so it wouldn't install. However, I didn't get any feedback telling me that it wasn't compatible, it just silently failed to install.

Comment 1

[Aakash Desai [:aakashd]]

Ted, can you send me a link of where that extension is? I don't see it on AMO :(.

Comment 2

[(not currently active) Ted Mielczarek]

It's not on AMO. I fixed the copy on my server, I'll recreate a broken copy and link it here shortly.

Comment 3

[(not currently active) Ted Mielczarek]

This should work:
http://people.mozilla.com/~tmielczarek/crashme.html

Comment 4

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

I noticed that without a hashcode, the install trigger seems to fail silently.
I still seem to have an issue with this particular plugin even with a hash code.

http://people.mozilla.com/~nhirata/html_tp/installcrashme.html

Comment 5

[Wesley Johnston (:wesj)]

Attachment: Fix

There is a secret property here we also need to check. I haven't tested it yet, but we can file a separate issue for your thing Naoki.

Comment 6

[Dave Townsend [:mossop]]

(In reply to comment #5)
> Created attachment 482389 [details] [diff] [review]
> Fix
> 
> There is a secret property here we also need to check. I haven't tested it yet,
> but we can file a separate issue for your thing Naoki.

s/secret/undocumented/ but now documented.

Comment 7

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

pushed:
http://hg.mozilla.org/mobile-browser/rev/759d42e8c02d

Comment 8

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Verified :

Mozilla/5.0 (Android; Linux armv71; rv2.0b8pre) Gecko/20101014 Firefox/4.0b8pre Fennec/4.0b2pre

Message is too long.  wrote bug : 604563