Firefox for Android Nightly 20120905 fails to update ("No updates found") if system language is not en-US

VERIFIED FIXED in Firefox 17

Status

()

Firefox for Android
General
--
major
VERIFIED FIXED
5 years ago
11 months ago

People

(Reporter: aryx, Assigned: snorp)

Tracking

({regression})

18 Branch
Firefox 18
ARM
Android
regression
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox17 fixed, firefox18 verified)

Details

Attachments

(2 attachments, 2 obsolete attachments)

Created attachment 659571 [details]
logcat for "no update"

Firefox for Android Nightly 20120905 fails to update ("No updates found").

Steps to reproduce:
1. Open about:firefox.
2. Tap the "Check for updates" button.

Actual result:
No update found.

Expected result:
Update found

ftp://ftp.mozilla.org/pub/mobile/nightly/latest-mozilla-central-android/ looks fine.

Logcat reads:
I/GeckoApp( 2118): Got message: Update:Check
I/GeckoApp( 2118): Return 
I/GeckoScreenshot( 2118): rect: 150.000000, 71.800003, 385.200012, 197.000000
I/UpdateService( 1762): next update will be at: Sun Sep 09 20:31:24 MESZ 2012
D/dalvikvm( 1762): GC_CONCURRENT freed 333K, 6% free 8324K/8775K, paused 14ms+5ms, total 39ms
I/UpdateService( 1762): no update available
Curious, are you running Android in a language other than English? Do you get an update if you switch your system language to English?
Assignee: nobody → snorp
Indeed, switching from German (de) to en-US in the Android system settings fixed this, so likely regression from bug 786380.
Summary: Firefox for Android Nightly 20120905 fails to update ("No updates found") → Firefox for Android Nightly 20120905 fails to update ("No updates found") if system language is not en-US

Updated

5 years ago
Blocks: 786380

Updated

5 years ago
Severity: normal → major
tracking-fennec: --- → ?
status-firefox18: --- → affected
I'm getting this as well. My nightly is also stuck on 2012-09-05. I believe I have Android set to en-US....but it might be en-CA. Where do I check this setting?
Android home screen > menu button > system settings > Category 'Personal' > Language & input > Language (first item).

But there are only en-US and en-UK available (at least for the Nexus S with Jelly Bean/4.1.1, don't know if this device dependent).
Ok, then I'm "English (Canada)"
Created attachment 659699 [details] [diff] [review]
use MOZ_UI_LOCALE in update URL instead of system locale on Android
Attachment #659699 - Flags: review?(mark.finkle)
Created attachment 659711 [details] [diff] [review]
always use en-US for update URL on Android
Attachment #659711 - Flags: review?(mark.finkle)
Attachment #659699 - Attachment is obsolete: true
Attachment #659699 - Flags: review?(mark.finkle)
Comment on attachment 659711 [details] [diff] [review]
always use en-US for update URL on Android

James found that the Gecko updater uses the locale stored in the update.locale file:
http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/update/nsUpdateService.js#852

We plan on trying to read that file too, from this JAR URL:
jar:jar:file:///data/app/org.mozilla.XXX.apk!/omni.ja!/update.locale
where XXX is the right fennec APK name.

We'll try using GeckoJarReader to load the text from the file
Attachment #659711 - Flags: review?(mark.finkle) → review-

Updated

5 years ago
Duplicate of this bug: 789926
Created attachment 659738 [details] [diff] [review]
Use contents of update.locale file in update URL on Android
Attachment #659738 - Flags: review?(mark.finkle)
Attachment #659711 - Attachment is obsolete: true
Comment on attachment 659738 [details] [diff] [review]
Use contents of update.locale file in update URL on Android

Looks good
Attachment #659738 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/bc546b6fe5b5

Comment 13

5 years ago
The old updater made it impossible to stay on the en-US build, when Android is on another language. What does this "update.locale" file contain?
In the end Fennec should stay on the original locale, no matter what, like desktop Firefox.
https://hg.mozilla.org/mozilla-central/rev/bc546b6fe5b5
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18

Comment 15

5 years ago
This is not going to fix existing users on nightly, right? We'll need to do some outreach to have folks re-install? Or should we try to get some update snippets that at least get folks on the multi build. Which may be better than having them stuck.

Updated

5 years ago
status-firefox18: affected → fixed
Keywords: regression
Version: Trunk → Firefox 18

Comment 16

5 years ago
As I said in Bug 789926 the issue was reproducible on Nightly 18.0a1 2012-09-09
HTC Desire Z (Android 2.3.3).

I can't reproduce it anymore on:
Nightly 18.0a1 (2012-09-11)
Device: HTC Desire Z
OS: Android 2.3.3
Status: RESOLVED → VERIFIED
status-firefox18: fixed → verified
How are we going to get users back on the nightly train?
(In reply to Chris AtLee [:catlee] from comment #17)
> How are we going to get users back on the nightly train?

IMO, the easiest way might be to use some redirection rules on the server to point all incoming requests to "en-US".

Thoughts?

Comment 19

5 years ago
(In reply to Chris AtLee [:catlee] from comment #17)
> How are we going to get users back on the nightly train?
An add-on hotfix?

Updated

5 years ago
Depends on: 790702

Comment 20

5 years ago
(In reply to Thomas Stache from comment #13)
> The old updater made it impossible to stay on the en-US build, when Android
> is on another language. What does this "update.locale" file contain?
> In the end Fennec should stay on the original locale, no matter what, like
> desktop Firefox.

After a automatic update the locale still changes from English to the system locale. Someone mentioned "multi builds". Can Fennec be configured via prefs to be English?

Comment 21

5 years ago
Setting the locale explicitly is bad for startup, so that's not available no more. There are single-locale builds for nightly and aurora, if you're keen to use fx in a specific language.

Comment 22

5 years ago
But is it okay for the updater to use the system locale code for the update check URL, or should it use the binary's locale (which is what I want)? Why does an automatic update *change* my locale?
(In reply to Axel Hecht [:Pike] from comment #21)
> There are single-locale builds for nightly and aurora, if you're keen
> to use fx in a specific language.
Do we really have single-locale builds for nightly and aurora? I can not find them.
http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-central-android-l10n/ and http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-central-android-l10n/ are empty and http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-central-android/ and http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-aurora-android/ contain only multi-locale builds.

Comment 24

5 years ago
What I download from nightly.mozilla.org runs in English, after one update the result runs in German. What's changing?

Comment 25

5 years ago
Grmpf, I can't find l10n nightlies and auroras, too, file a bug, please?
Here are the locales with more than 100 hits since sept 5th:

da-DK 100
es-MX 101
fi-FI 103
pt-PT 106
sk 109
de-CH 125
de-AT 169
ja 182
fr 192
es-US 213
ko-KR 264
pl-PL 280
en-AU 296
sv-SE 307
nl-NL 308
pt-BR 364
it-IT 488
en-CA 582
de 656
ru-RU 700
fr-FR 991
es-ES 1009
ru 1691
ja-JP 1782
en-GB 2824
de-DE 2953
zh-TW 4196
zh-CN 23951
en-US 9391999

I've added symlinks for en-CA, en-GB and en-AU to en-US (which is the multilocale build). Which other locales should be updated to the latest en-US (multilocale) build?

Comment 27

5 years ago
(In reply to Thomas Stache from comment #24)
> What I download from nightly.mozilla.org runs in English, after one update
> the result runs in German. What's changing?

I think nightly.mozilla.org is pointing to en-US.
http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-central-android/en-US/fennec-18.0a1.en-US.android-arm.apk

The link should point to the multi:
http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-central-android/fennec-18.0a1.multi.android-arm.apk

That means you're installing en-US, which is getting updated to the multilocale.  We should start off pointing at multi.

Comment 28

5 years ago
https://bugzilla.mozilla.org/show_bug.cgi?id=791482

Updated

5 years ago
Blocks: 791387

Comment 29

5 years ago
I guess we can put stranded folks on the multi by default. Also +1 to make that the default offering on nightly.m.o.

Is there a bug about the en-US becoming multi on update?
Comment on attachment 659738 [details] [diff] [review]
Use contents of update.locale file in update URL on Android

[Approval Request Comment]
Necessary dependency for bug 786380
Attachment #659738 - Flags: approval-mozilla-aurora?

Comment 31

5 years ago
(In reply to Axel Hecht [:Pike] from comment #29)
> I guess we can put stranded folks on the multi by default. Also +1 to make
> that the default offering on nightly.m.o.
> 
> Is there a bug about the en-US becoming multi on update?

Currently we're using the en-US update channel as the multilocale channel for any products/branches that have multilocale builds; if we want to separate the two, we'd have to support a separate multilocale channel.
I'm going to symlink the following locales to en-US as well:

it-IT -> en-US 488
pl-PL -> en-US 280
ja-JP -> en-US 1782
de-CH -> en-US 125
nl-NL -> en-US 308
de-AT -> en-US 169
da-DK -> en-US 100
ru-RU -> en-US 700
fi-FI -> en-US 103
ko-KR -> en-US 264
de-DE -> en-US 2953
fr-FR -> en-US 991

These users currently aren't getting updates, and we have no way of knowing if they were on a single locale build or the multilocale build before. The counts are the # of hits to AUS for that locale from sept 5-12.
(In reply to James Willcox (:snorp) (jwillcox@mozilla.com) from comment #30)
> Comment on attachment 659738 [details] [diff] [review]
> Use contents of update.locale file in update URL on Android
> 
> [Approval Request Comment]
> Necessary dependency for bug 786380

We'll discuss bug 786380 tomorrow, and a decision about uplift of this bug will happen at the same time.

Updated

5 years ago
Attachment #659738 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

Comment 34

5 years ago
Pushed to Aurora: http://hg.mozilla.org/releases/mozilla-aurora/rev/a62178800150
status-firefox17: --- → fixed
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.