Closed Bug 789791 Opened 8 years ago Closed 8 years ago

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

Categories

(Firefox for Android :: General, defect, major)

18 Branch
ARM
Android
defect
Not set
major

Tracking

()

VERIFIED FIXED
Firefox 18
Tracking Status
firefox17 --- fixed
firefox18 --- verified

People

(Reporter: aryx, Assigned: snorp)

References

Details

(Keywords: regression)

Attachments

(2 files, 2 obsolete files)

Attached file 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
Blocks: 786380
Severity: normal → major
tracking-fennec: --- → ?
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)"
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-
Duplicate of this bug: 789926
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+
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
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
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.
Keywords: regression
Version: Trunk → Firefox 18
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
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?
(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?
Depends on: 790702
(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?
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.
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.
What I download from nightly.mozilla.org runs in English, after one update the result runs in German. What's changing?
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?
(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.
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?
(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.
Attachment #659738 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.