Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 289108 - www-client/icecat-3.5.6: "About" window broken for non-english locales
Summary: www-client/icecat-3.5.6: "About" window broken for non-english locales
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Mozilla Gentoo Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-10-14 20:56 UTC by Fiona Klute
Modified: 2010-02-06 13:19 UTC (History)
6 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Fiona Klute 2009-10-14 20:56:20 UTC
Icecat's "About" window is broken for non-English locales. I installed support for German, Japanese and English and tested with de_DE.UTF-8, ja_JP.UTF-8, en_US.UTF-8 and C locale settings. The de_DE.UTF-8 and ja_JP.UTF-8 locales show the same behavior: When clicking the "About" entry in the "Help" menu, the appearing window contains an error message from the XML-parser. The German one looks like this:

 XML-Verarbeitungsfehler: Nicht definierte Entität
 Adresse: chrome://browser/content/aboutDialog.xul
 Zeile Nr. 40, Spalte 9:        &logoCopyright;
                        --------^

As I can't to reproduce the error with English locale settings, I'm unable to
provide an English error message. Typing "about:" in the location bar works in any locale. Below my attempt to translate the error message to English:

 XML-Parser error: Undefined entity
 Location: chrome://browser/content/aboutDialog.xul
 Row 40, Column 9:        &logoCopyright;
                  --------^
Comment 1 Nikolaj Šujskij 2009-10-25 21:46:20 UTC
icecat hit main Portage tree, but the bug persists.
Confirmed on ru_RU-system
Comment 2 Jory A. Pratt gentoo-dev 2009-10-25 21:59:41 UTC
(In reply to comment #1)
> icecat hit main Portage tree, but the bug persists.
> Confirmed on ru_RU-system
> 

I am aware of the bug and will continue to work on it.
Comment 3 Rafał Mużyło 2009-10-26 16:13:40 UTC
Confirming for pl_PL.UTF-8

Comes from missing ENTITY definition in
locale/browser/aboutDialog.dtd in ${lang}.jar
for logoCopyright - for en_US it's simply:
<!ENTITY logoCopyright              "">
Once added, dialog shows correctly.
Comment 4 Tim O'Kelly 2009-11-10 11:40:42 UTC
Confirmed for ru_RU locale
Comment 5 Raffaello D. Di Napoli 2009-11-15 12:02:56 UTC
Confirmed on it_IT locale.
Comment 6 Rafał Mużyło 2010-01-10 23:45:06 UTC
The reason for this bug is obvious:
in src_prepare, block starting with "for X in ${linguas}; do"
has been simplified too much, when compared to
http://gnuzilla.gnu.org/download/langpacks/IceCatTranslate-0.1.py

On the other hand, that script seems to do bit too much for Gentoo.
A balance needs to be found.
Comment 7 Harald van Dijk (RETIRED) gentoo-dev 2010-02-06 01:32:45 UTC
Downloading the rebranded Icecat xpi files instead of the Firefox ones, and dropping the ebuild's own rebranding seems like the simplest solution, but...

(In reply to comment #6)
> On the other hand, that script seems to do bit too much for Gentoo.
> A balance needs to be found.

do you really mean it does too much, or just that it does too much to be suitable for an ebuild? If the former, those xpis shouldn't be used, but if the latter, then that's not a problem since it wouldn't be done from the ebuild.
Comment 8 Rafał Mużyło 2010-02-06 02:27:44 UTC
What I'm really disagreeing about is the current icecat ebuild.
Over 99% of the code is the same between firefox and icecat tarballs,
yet we're still using separate tarballs.
Sure, doing it another way would require a bit more work from
the maintainer for the first time, but we could simply add
a little git repository with produced icons
and later just update it with changes from gnuzilla svn.

That would reduce bandwidth for both mirrors and icecat users
by nearly a half, as firefox tarball is needed any way for
xulrunner.

As for the script, you should notice, that among other,
it downloads the requested xpi file from mozilla mirror.

I would drop (or reduce to english only) that part which
differs between locales, even though  my language surprisingly
is there.
Comment 9 Harald van Dijk (RETIRED) gentoo-dev 2010-02-06 08:53:11 UTC
(In reply to comment #8)
> What I'm really disagreeing about is the current icecat ebuild.
> Over 99% of the code is the same between firefox and icecat tarballs,
> yet we're still using separate tarballs.

The problem with that isn't that firefox and icecat use different tarballs, but that firefox and xulrunner use the same tarball, and icecat follows firefox. If a newer standalone xulrunner is released (and there's an open upstream bug about it), then it might be possible to drop the xulrunner bits from the firefox and icecat tarballs, so that they would be significantly smaller.

> [...]
> As for the script, you should notice, that among other,
> it downloads the requested xpi file from mozilla mirror.
> 
> I would drop (or reduce to english only) that part which
> differs between locales, even though  my language surprisingly
> is there.

The "Modified by"? Sure, that's not necessary, but if that's all, the xpi files created by it seem fine, so it would be just as easy to just download those directly.
Comment 10 Jory A. Pratt gentoo-dev 2010-02-06 12:49:10 UTC
Harald feel free to fix any bugs that arise for icecat, I am extremely busy with other tasks at the moment.
Comment 11 Harald van Dijk (RETIRED) gentoo-dev 2010-02-06 13:19:36 UTC
Sure, done.