764300 - ko and pt-BR do not work on Firefox 14 beta 7

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Adrian Tamas (:AdrianT)]

Firefox Mobile Native 14 beta 7/ Nightly 16.0a1 2012-06-12
Device: Galaxy Nexus (Android 4.0.2)

Steps to reproduce:
1. Go to settings and set the language to pt-Br or Ko.
2. Confirm changes.
3. Open Firefox Beta and visually inspect the about pages and app menus.

Expected results:
All strings are localized. Language packs added through Bug 760169.

Actual results:
All strings are in English.

Notes:
The issue is not reproducible on Aurora 15.0a2 2012-06-12

Comment 1

[Axel Hecht [:Pike]]

So, the l10n-changesets in buildbot-configs confirm that neither ko nor pt-BR are in the multi-locale build. CCing nthomas who pushed that config.

That's odd, as the change to maemo-locales that should have triggered this was pushed before the release configs:

http://hg.mozilla.org/releases/mozilla-beta/pushloghtml?changeset=bb7ea7f62a6b, Mon Jun 11 19:33
http://hg.mozilla.org/build/buildbot-configs/pushloghtml?changeset=0c515157e7b9, Tue Jun 12 15:54:41

Comment 2

[Nick Thomas [:nthomas] (UTC+12)]

I'm not sure how I can help here. I pulled the l10n changesets using 
https://l10n.mozilla.org/shipping/json-changesets?ms=fennec14_beta_b7&platforms=android&multi_android-multilocale_repo=mobile-browser&multi_android-multilocale_rev=default&multi_android-multilocale_path=locales%2Fmaemo-locales

and that doesn't mark ko or pt-BR with "android-multilocale". However I don't know the mobile release automation well enough to know what controls what locales are put into the multilocale build.

Comment 3

[Nick Thomas [:nthomas] (UTC+12)]

From a quick look at the code that generates the multi-locale build it is keying off the platforms in 
http://hg.mozilla.org/build/buildbot-configs/file/FENNEC_14_0b7_RELEASE/mozilla/l10n-changesets_mobile-beta.json
so the question is why ko and pt-BR aren't marked android-multilocale.

Comment 4

[Alex Keybl [:akeybl]]

As a heads up, this will likely require a quick re-spin of FN14b7 once resolved. I'm including Axel and Aki here in case they can shed some light.

Comment 5

[Axel Hecht [:Pike]]

Yes, that's the wrong URL, sadly. 

https://l10n.mozilla.org/shipping/json-changesets?ms=fennec14_beta_b7&platforms=android&multi_android-multilocale_repo=releases/mozilla-beta&multi_android-multilocale_rev=default&multi_android-multilocale_path=mobile/android/locales/maemo-locales is the right one, note the mozilla-beta repo and the changed path.

Bug 762531 is filed to update teh defaults.

Comment 6

[Aki Sasaki (not active)]

Ugh.  Filing a bug to update the defaults was at the back of my mind but never got around to doing it.  It never bit us before because we've kept the same multilocale list since the beginning.  Sorry Nick.

Comment 7

[Aki Sasaki (not active)]

This should be fixed in build 2, as well as going forward if we make sure to override the l10n dashboard defaults.

Our documentation shows what needs to change:
https://wiki.mozilla.org/Release:Release_Automation_on_Mercurial:Configuration#L10N_Changesets

bug 762531 tracks making the l10n dashboard defaults useful again.

I think we can close this bug to verify in build 2.

Comment 8

[Aki Sasaki (not active)]

Oops, didn't mean to reset the blocking-fennec1.0 flag.

Comment 9

[Aaron Train [:aaronmt]]

Verified ko/pt-BR included and functional in 14.0b7-build2