Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

False positive in L10n dashboard?

16 views
Skip to first unread message

Ricardo Palomares Martí­nez

unread,
Aug 15, 2014, 3:48:50 PM8/15/14
to
Hi,

I've just noticed that es-ES Fennec Aurora shows 2 errors:

https://l10n.mozilla.org/dashboard/compare?run=421516

But, when I compare the es-ES file:

http://hg.mozilla.org/releases/l10n/mozilla-aurora/es-ES/file/2a2ae658bb39/mobile/android/base/android_strings.dtd#l110

with the en-US one:

http://hg.mozilla.org/releases/mozilla-aurora/file/38a71dd3290c/mobile/android/base/locales/en-US/android_strings.dtd#l152

I can't see why es-ES is wrong and en-US is right.

Is that a false positive in compare-locales?

TIA

--
Ricardo Palomares (RickieES)
http://www.mozilla-hispano.org/
http://www.proyectonave.es/
https://diasp.eu/u/rickiees

Sebastian Hengst

unread,
Aug 15, 2014, 4:01:09 PM8/15/14
to Ricardo Palomares Martí­nez, dev-...@lists.mozilla.org
Hi,

that warning surfaced after compare-locales added more tests. en-US is
also affected. The issue will be fixed in Firefox Mobile 34, see
https://bugzilla.mozilla.org/show_bug.cgi?id=1048052

If you want to get rid of the error now, use \u0022 instead of "
and other constructs.

Sebastian

-------- Original-Nachricht --------
Betreff: False positive in L10n dashboard?
Von: Ricardo Palomares Martí­nez <rpm....@proyectonave.es>
Datum: 2014-08-15 21:48

Ricardo Palomares Martí­nez

unread,
Aug 15, 2014, 4:08:30 PM8/15/14
to
El 15/08/14 22:01, Sebastian Hengst escribió:
> Hi,
>
> that warning surfaced after compare-locales added more tests. en-US is
> also affected. The issue will be fixed in Firefox Mobile 34, see
> https://bugzilla.mozilla.org/show_bug.cgi?id=1048052
>
> If you want to get rid of the error now, use \u0022 instead of &quot;
> and other constructs.


Oh, OK, I've just checked in central and both en-US and es-ES use the
escape sequences. I'd rather keep the localization consistent with
en-US in each channel, even if that means keeping the errors in
compare-locales.

Thank you for your quick answer.

Mad Maks

unread,
Aug 20, 2014, 3:57:59 PM8/20/14
to
Op 08/15/2014 om 10:08 PM schreef Ricardo Palomares Mart�nez:
> El 15/08/14 22:01, Sebastian Hengst escribi�:
>> Hi,
>>
>> that warning surfaced after compare-locales added more tests. en-US is
>> also affected. The issue will be fixed in Firefox Mobile 34, see
>> https://bugzilla.mozilla.org/show_bug.cgi?id=1048052
>>
>> If you want to get rid of the error now, use \u0022 instead of &quot;
>> and other constructs.
>
>
> Oh, OK, I've just checked in central and both en-US and es-ES use the
> escape sequences. I'd rather keep the localization consistent with
> en-US in each channel, even if that means keeping the errors in
> compare-locales.
>
> Thank you for your quick answer.
>
>

I notice a same 'problem' :

devtools/debugger.properties
unknown escape sequence, \ at line 305, column 31 for
watchExpressionsSeparatorLabel

but de nl version is the same as en-US

greetings

Tim Maks

Francesco Lodolo [:flod]

unread,
Aug 21, 2014, 2:25:52 AM8/21/14
to dev-...@lists.mozilla.org
Il 20/08/14 21:57, Mad Maks ha scritto:
> devtools/debugger.properties
> unknown escape sequence, \ at line 305, column 31 for
> watchExpressionsSeparatorLabel
>
> but de nl version is the same as en-US
English string uses "\ " to escape the starting space
http://hg.mozilla.org/releases/mozilla-aurora/file/default/browser/locales/en-US/chrome/browser/devtools/debugger.properties#l305

watchExpressionsSeparatorLabel=\ →

You can work around the warning by replacing "\ →" with "\u0020→" (same
result, blank space before →).

Francesco

0 new messages