[ANNOUNCE] compare-locales 0.9 now running on the dashboard

classic Classic list List threaded Threaded
9 messages Options
Reply | Threaded
Open this post in threaded view
|

[ANNOUNCE] compare-locales 0.9 now running on the dashboard

Axel Hecht
Hi,

a few days have gone by and nobody yelled at me for 0.9, so I've decided
to just add it to l10n-stage-sj, aka the dashboard.

The infrastructure is currently digging through the 1.9.1 statuses, and
then I'll switch the automation back on.

Here's what you'll need to deal with from now on:

More Errors:
The new version is detecting more errors than the previous versions did.
These are things that you have to fix to make the browser work. The
count of errors needs to go down to zero.

Warnings:
This version comes back with a new breed of report data, Warnings. These
are occurrences that the parser tripped over, and they may or may not be
bugs that you have to fix. They may or may not crash the browser.
Basically, I can't tell up-front, so it'll be called out, and you need
to have a look at it. I'd call out entity references here in particular.
A '©' reference is fine in xhtml, but not in xul. We can't tell.
What the code checks for is, is the localization referencing any
entities that the en-US code doesn't. Yeah, I assume that en-US works.
You'll need to make an educated guess on whether the entity it's warning
about will lead to a yellow screen of death, or if it's going to be just
fine. Or try it out.

Bottom line, it's OK to have the warnings count not go down to 0. The
errors count needs to go to 0, though.

Use this thread or the one opened by flod to ask questions, or trade
tips and tricks to fix particular issues.

Axel
_______________________________________________
dev-l10n mailing list
[hidden email]
https://lists.mozilla.org/listinfo/dev-l10n
Reply | Threaded
Open this post in threaded view
|

Re: [ANNOUNCE] compare-locales 0.9 now running on the dashboard

Simon Paquet-2
Axel Hecht wrote on 13. Oct 2010:

> Here's what you'll need to deal with from now on:
>
> More Errors:
> The new version is detecting more errors than the previous versions
> did. These are things that you have to fix to make the browser work.
> The count of errors needs to go down to zero.
>
> Warnings:
> This version comes back with a new breed of report data, Warnings.
> These are occurrences that the parser tripped over, and they may or
> may not be bugs that you have to fix. They may or may not crash the
> browser. Basically, I can't tell up-front, so it'll be called out,
> and you need to have a look at it. I'd call out entity references
> here in particular.
>
> Bottom line, it's OK to have the warnings count not go down to 0. The
> errors count needs to go to 0, though.

Does that mean, that you won't accept locale sign-offs where the locale
still has open errors of the new kind?

--
Simon
_______________________________________________
dev-l10n mailing list
[hidden email]
https://lists.mozilla.org/listinfo/dev-l10n
Reply | Threaded
Open this post in threaded view
|

Re: [ANNOUNCE] compare-locales 0.9 now running on the dashboard

Axel Hecht
On 14.10.10 09:42, Simon Paquet wrote:

> Axel Hecht wrote on 13. Oct 2010:
>
>> Here's what you'll need to deal with from now on:
>>
>> More Errors:
>> The new version is detecting more errors than the previous versions
>> did. These are things that you have to fix to make the browser work.
>> The count of errors needs to go down to zero.
>>
>> Warnings:
>> This version comes back with a new breed of report data, Warnings.
>> These are occurrences that the parser tripped over, and they may or
>> may not be bugs that you have to fix. They may or may not crash the
>> browser. Basically, I can't tell up-front, so it'll be called out, and
>> you need to have a look at it. I'd call out entity references here in
>> particular.
>> Bottom line, it's OK to have the warnings count not go down to 0. The
>> errors count needs to go to 0, though.
>
> Does that mean, that you won't accept locale sign-offs where the locale
> still has open errors of the new kind?
>

That means I hope to not run into that situation :-)
For the fx4 beta sign-offs, I'll make a judgment call as I do on any
non-perfect sign-off. For releases, I'll be tough.

Axel
_______________________________________________
dev-l10n mailing list
[hidden email]
https://lists.mozilla.org/listinfo/dev-l10n
Reply | Threaded
Open this post in threaded view
|

Re: [ANNOUNCE] compare-locales 0.9 now running on the dashboard

Jesper Kristensen-4
In reply to this post by Axel Hecht
Den 13-10-2010 19:31, Axel Hecht skrev:

> Warnings:
> This version comes back with a new breed of report data, Warnings. These
> are occurrences that the parser tripped over, and they may or may not be
> bugs that you have to fix. They may or may not crash the browser.
> Basically, I can't tell up-front, so it'll be called out, and you need
> to have a look at it. I'd call out entity references here in particular.
> A '©' reference is fine in xhtml, but not in xul. We can't tell.
> What the code checks for is, is the localization referencing any
> entities that the en-US code doesn't. Yeah, I assume that en-US works.
> You'll need to make an educated guess on whether the entity it's warning
> about will lead to a yellow screen of death, or if it's going to be just
> fine. Or try it out.

I think it would be safe to assume that <, > " and ' are
always available in DTD files. Some English strings in Thunderbird use
apostrophe for citations. That is not valid in Danish, so we have
changed them to quotation marks, which must be written as ". Having
bogus warnings produced by compare-locale is annoying and decreases the
value of the output.
_______________________________________________
dev-l10n mailing list
[hidden email]
https://lists.mozilla.org/listinfo/dev-l10n
Reply | Threaded
Open this post in threaded view
|

Re: [ANNOUNCE] compare-locales 0.9 now running on the dashboard

Anas Husseini-2
On Thu, Oct 14, 2010 at 5:16 PM, Jesper Kristensen <
[hidden email]> wrote:

> Having bogus warnings produced by compare-locale is annoying and decreases
> the value of the output.
>
>
Then perhaps implementing an --ignore option to compare-locales (e.g.
--ignore=references) would solve this annoyance?

Anas
_______________________________________________
dev-l10n mailing list
[hidden email]
https://lists.mozilla.org/listinfo/dev-l10n
Reply | Threaded
Open this post in threaded view
|

Re: [ANNOUNCE] compare-locales 0.9 now running on the dashboard

Axel Hecht
In reply to this post by Jesper Kristensen-4
On 14.10.10 17:16, Jesper Kristensen wrote:

> Den 13-10-2010 19:31, Axel Hecht skrev:
>> Warnings:
>> This version comes back with a new breed of report data, Warnings. These
>> are occurrences that the parser tripped over, and they may or may not be
>> bugs that you have to fix. They may or may not crash the browser.
>> Basically, I can't tell up-front, so it'll be called out, and you need
>> to have a look at it. I'd call out entity references here in particular.
>> A '&copy;' reference is fine in xhtml, but not in xul. We can't tell.
>> What the code checks for is, is the localization referencing any
>> entities that the en-US code doesn't. Yeah, I assume that en-US works.
>> You'll need to make an educated guess on whether the entity it's warning
>> about will lead to a yellow screen of death, or if it's going to be just
>> fine. Or try it out.
>
> I think it would be safe to assume that &lt;, &gt; &quot; and &apos; are
> always available in DTD files. Some English strings in Thunderbird use
> apostrophe for citations. That is not valid in Danish, so we have
> changed them to quotation marks, which must be written as &quot;. Having
> bogus warnings produced by compare-locale is annoying and decreases the
> value of the output.

Ouch, yeah, that's a bug straight,
https://bugzilla.mozilla.org/show_bug.cgi?id=604404.

XML entities are of course totally fine.

Axel
_______________________________________________
dev-l10n mailing list
[hidden email]
https://lists.mozilla.org/listinfo/dev-l10n
Reply | Threaded
Open this post in threaded view
|

Re: [ANNOUNCE] compare-locales 0.9 now running on the dashboard

Rimas Kudelis
In reply to this post by Jesper Kristensen-4
2010.10.14 18:16, Jesper Kristensen rašė:

> Den 13-10-2010 19:31, Axel Hecht skrev:
>> Warnings:
>> This version comes back with a new breed of report data, Warnings. These
>> are occurrences that the parser tripped over, and they may or may not be
>> bugs that you have to fix. They may or may not crash the browser.
>> Basically, I can't tell up-front, so it'll be called out, and you need
>> to have a look at it. I'd call out entity references here in particular.
>> A '&copy;' reference is fine in xhtml, but not in xul. We can't tell.
>> What the code checks for is, is the localization referencing any
>> entities that the en-US code doesn't. Yeah, I assume that en-US works.
>> You'll need to make an educated guess on whether the entity it's warning
>> about will lead to a yellow screen of death, or if it's going to be just
>> fine. Or try it out.
>
> I think it would be safe to assume that &lt;, &gt; &quot; and &apos; are
> always available in DTD files. Some English strings in Thunderbird use
> apostrophe for citations. That is not valid in Danish, so we have
> changed them to quotation marks, which must be written as &quot;. Having
> bogus warnings produced by compare-locale is annoying and decreases the
> value of the output.

as a side note, why didn't you use curly quotes?

Rimas
_______________________________________________
dev-l10n mailing list
[hidden email]
https://lists.mozilla.org/listinfo/dev-l10n
Reply | Threaded
Open this post in threaded view
|

Re: [ANNOUNCE] compare-locales 0.9 now running on the dashboard

Jesper Kristensen-4
Den 14-10-2010 21:38, Rimas Kudelis skrev:

> 2010.10.14 18:16, Jesper Kristensen rašė:
>> Den 13-10-2010 19:31, Axel Hecht skrev:
>>> Warnings:
>>> This version comes back with a new breed of report data, Warnings. These
>>> are occurrences that the parser tripped over, and they may or may not be
>>> bugs that you have to fix. They may or may not crash the browser.
>>> Basically, I can't tell up-front, so it'll be called out, and you need
>>> to have a look at it. I'd call out entity references here in particular.
>>> A '&copy;' reference is fine in xhtml, but not in xul. We can't tell.
>>> What the code checks for is, is the localization referencing any
>>> entities that the en-US code doesn't. Yeah, I assume that en-US works.
>>> You'll need to make an educated guess on whether the entity it's warning
>>> about will lead to a yellow screen of death, or if it's going to be just
>>> fine. Or try it out.
>>
>> I think it would be safe to assume that &lt;, &gt; &quot; and &apos; are
>> always available in DTD files. Some English strings in Thunderbird use
>> apostrophe for citations. That is not valid in Danish, so we have
>> changed them to quotation marks, which must be written as &quot;. Having
>> bogus warnings produced by compare-locale is annoying and decreases the
>> value of the output.
>
> as a side note, why didn't you use curly quotes?
>
> Rimas

Hmm. Good point.
_______________________________________________
dev-l10n mailing list
[hidden email]
https://lists.mozilla.org/listinfo/dev-l10n
Reply | Threaded
Open this post in threaded view
|

Re: [ANNOUNCE] compare-locales 0.9 now running on the dashboard

Axel Hecht
In reply to this post by Axel Hecht
On 14.10.10 18:39, Axel Hecht wrote:

> On 14.10.10 17:16, Jesper Kristensen wrote:
>> Den 13-10-2010 19:31, Axel Hecht skrev:
>>> Warnings:
>>> This version comes back with a new breed of report data, Warnings. These
>>> are occurrences that the parser tripped over, and they may or may not be
>>> bugs that you have to fix. They may or may not crash the browser.
>>> Basically, I can't tell up-front, so it'll be called out, and you need
>>> to have a look at it. I'd call out entity references here in particular.
>>> A '&copy;' reference is fine in xhtml, but not in xul. We can't tell.
>>> What the code checks for is, is the localization referencing any
>>> entities that the en-US code doesn't. Yeah, I assume that en-US works.
>>> You'll need to make an educated guess on whether the entity it's warning
>>> about will lead to a yellow screen of death, or if it's going to be just
>>> fine. Or try it out.
>>
>> I think it would be safe to assume that &lt;, &gt; &quot; and &apos; are
>> always available in DTD files. Some English strings in Thunderbird use
>> apostrophe for citations. That is not valid in Danish, so we have
>> changed them to quotation marks, which must be written as &quot;. Having
>> bogus warnings produced by compare-locale is annoying and decreases the
>> value of the output.
>
> Ouch, yeah, that's a bug straight,
> https://bugzilla.mozilla.org/show_bug.cgi?id=604404.
>
> XML entities are of course totally fine.
>

FIXED, not yet run through all locales, just for Danish on central and
1.9.2.

Gonna release an 0.9.1 soon. Gonna give this a few cycles.

Axel
_______________________________________________
dev-l10n mailing list
[hidden email]
https://lists.mozilla.org/listinfo/dev-l10n