630475 - Maemo single locale deb / build doesn't start due missing link to chrome/localized.manifest

Status: RESOLVED WORKSFORME

(Firefox for Android Graveyard :: General, defect)

Description

[ahoza]

Attachment: Screenshot

Device: Nokia N900
Builds: 
- 20110131 build from the following location: http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-central-maemo5-gtk-l10n/ro/
- release candidate builds from the following location: http://ftp.mozilla.org/pub/mozilla.org/mobile/candidates/4.0b4-candidates/build1/maemo5-gtk/, locales: fr, fi, de

Steps to reproduce:

1. Uninstall Fennec and delete profiles (if it exists)
2. Download and install a build from the above locations
3. Start Fennec.

Expected results:
Fennec starts with about:firstrun.

Actual results:
error instead of the main window
> <window id="main-window" (see attached screenshot) 

This is a follow up of bug 605411, which was marked Resolved Fixed.

Comment 1

[Aki Sasaki (not active)]

http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-central-maemo5-gtk-l10n/fennec-4.0b5pre.ro.linux-gnueabi-arm.tar.bz2 wfm

http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-central-maemo5-gtk-l10n/ro/fennec_4.0~b5~20110208004541_armel.deb does not.

Something's wrong with deb packaging.

Comment 2

[Aki Sasaki (not active)]

Recursive checksum diff between en-US deb and ro deb.

deathduck:~/wrk/n900-l10n [13:56:57]
$ diff en-US_deb.txt ro_deb.txt | more
4d3
< 18800 396 ./opt/mozilla/fennec-4.0b5pre/chrome/en-US.jar
7a7,8
> 54544 417 ./opt/mozilla/fennec-4.0b5pre/chrome/ro.jar
> 20206 2 ./opt/mozilla/fennec-4.0b5pre/chrome/ro.manifest
84c85
< 48237 1 ./opt/mozilla/fennec-4.0b5pre/crashreporter-override.ini
---
> 27218 1 ./opt/mozilla/fennec-4.0b5pre/crashreporter-override.ini
86c87
< 341 4 ./opt/mozilla/fennec-4.0b5pre/crashreporter.ini
---
> 45084 5 ./opt/mozilla/fennec-4.0b5pre/crashreporter.ini
91a93
> 63583 1 ./opt/mozilla/fennec-4.0b5pre/defaults/preferences/mobile-l10n.js
93,94d94
< 33541 3 ./opt/mozilla/fennec-4.0b5pre/dictionaries/en-US.aff
< 52145 610 ./opt/mozilla/fennec-4.0b5pre/dictionaries/en-US.dic
226c226
< 26712 1 ./opt/mozilla/fennec-4.0b5pre/update.locale
---
> 94 1 ./opt/mozilla/fennec-4.0b5pre/update.locale
228c228
< 29959 1 ./opt/mozilla/fennec-4.0b5pre/updater.ini
---
> 46359 1 ./opt/mozilla/fennec-4.0b5pre/updater.ini

Comment 3

[Aki Sasaki (not active)]

Do we need a dictionaries/ro.aff and ro.dic ?

Comment 4

[Axel Hecht [:Pike]]

I thought that we don't want to ship dictionaries even for en-US.

I'm missing the removed en-US.manifest.

Comment 5

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

I think we remove the dictionaries during packaging:

http://mxr.mozilla.org/mobile-browser/source/locales/Makefile.in#105

Comment 6

[Aki Sasaki (not active)]

I'm not sure how to proceed here.
Does someone else know what's going on and what needs to be fixed?

Comment 7

[Axel Hecht [:Pike]]

The diff you pasted says that en-US.manifest is still in there, but the jar is not. Somewhere there would be my suspect #1.

Comment 8

[Aki Sasaki (not active)]

[09:30]	<Pike>	aki: can you look in the chrome.manifest of omni.jar if it references en-US? also, what's in mobile-l10n.js, the prefs file? and does ro.manifest reference stuff that exists?

Spot on.

chrome.manifest is only a manifest-of-manifests, but it references localized.manifest, which is all en-US:


deathduck:~/wrk/n900-l10n [09:37:03]
$ more ro_deb/opt/mozilla/fennec-4.0b5pre/chrome/localized.manifest 
# en-US.manifest
locale global-region en-US jar:en-US.jar!/locale/en-US/global-region/
locale global-platform en-US jar:en-US.jar!/locale/en-US/global-platform/
override chrome://global/locale/netError.dtd    chrome://browser/locale/netError.dtd
locale autoconfig en-US jar:en-US.jar!/locale/en-US/autoconfig/
<snip many more lines of en-US>


And the prefs file is in the wrong directory:


deathduck:~/wrk/n900-l10n [09:38:26]
$ find ro_deb -name mobile\*.js
ro_deb/opt/mozilla/fennec-4.0b5pre/defaults/pref/mobile-l10n.js
ro_deb/opt/mozilla/fennec-4.0b5pre/defaults/pref/mobile.js
ro_deb/opt/mozilla/fennec-4.0b5pre/defaults/preferences/mobile-l10n.js
deathduck:~/wrk/n900-l10n [09:38:31]
$ cat ro_deb/opt/mozilla/fennec-4.0b5pre/defaults/pref/mobile-l10n.js
//@line 36 "/home/cltbld/build/mobile-trunk-maemo5-gtk-nightly/mozilla-central/mobile/locales/en-US/mobile-l10n.js"

//@line 38 "/home/cltbld/build/mobile-trunk-maemo5-gtk-nightly/mozilla-central/mobile/locales/en-US/mobile-l10n.js"

pref("general.useragent.locale", "en-US");
deathduck:~/wrk/n900-l10n [09:38:44]
$ cat ro_deb/opt/mozilla/fennec-4.0b5pre/defaults/preferences/mobile-l10n.js
//@line 36 "/home/cltbld/build/mobile-trunk-maemo5-gtk-l10n/l10n-central/ro/mobile/mobile-l10n.js"

//@line 38 "/home/cltbld/build/mobile-trunk-maemo5-gtk-l10n/l10n-central/ro/mobile/mobile-l10n.js"

pref("general.useragent.locale", "ro");

Comment 9

[Aki Sasaki (not active)]

Attachment: this may fix the prefs dir issue

Untested. May fix this entire bug; not sure, I'd have to repack a deb with the mobile-l10n.js file in the right directory.

Comment 10

[Aki Sasaki (not active)]

Copying the mobile-l10n.js into defaults/pref didn't fix it.
We probably need to update localized.manifest to be non-en-US as well.

Comment 11

[Ioana Chiorean]

Still reproducible with release candidate build 1 located at:

http://ftp.mozilla.org/pub/mozilla.org/mobile/candidates/4.0b5-candidates/build2/maemo5-gtk/

Comment 12

[Ioana Chiorean]

Attachment: new error

Comment 13

[Ioana Chiorean]

the builds are broken also for build 3 :  http://ftp.mozilla.org/pub/mozilla.org/mobile/candidates/4.0b5-candidates/build3/maemo5-gtk/ 

please see the attached new error.

Comment 14

[Aki Sasaki (not active)]

Maemo is now tier 3, and we stopped running both Maemo builds and single locale repacks quite a while ago.

I recommend closing this bug.