Jonathan Kew
unread,Oct 4, 2012, 6:36:01 AM10/4/12You do not have permission to delete messages in this group
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to flod, Alex Keybl, sri...@mozilla.com, Sheila Mooney, David Keeler, Axel Hecht, sn...@snorp.net, dev-...@lists.mozilla.org, Release Management Team
On 4/10/12 05:58, flod wrote:
> Il 03/10/12 22:27, Alex Keybl ha scritto:
>> Good to know - who's going to take point to make sure that developers
>> know this in the future? I trust you'll file a bug if this is very
>> disruptive and needs to be fixed for FF17/18.
>>
> At this point in the cycle I would leave Aurora's repository alone (Axel?).
>
> This document was written some months ago, but the "rule" has always
> been there
>
https://developer.mozilla.org/en-US/docs/Making_String_Changes [*]
>
> Unfortunately it seems that a lot of people are not aware of the right
> behavior lately. This is the last example:
>
https://bugzilla.mozilla.org/show_bug.cgi?id=797036#c6
Looking at the patch there and the string changes it made, ISTM that the
proper action would be to back out that bug and post a corrected patch
for re-review with appropriate changes to entity names.
Maybe someone closer to that area of the code/product should take
another look, and decide how to proceed...
JK
>
> If that's ok for developers (it's not, but let's say it is), I think
> that at least reviewers should be a lot more careful, in particular when
> changing string days before merge (the mess goes into Aurora and then
> you'll have to break string freeze to fix it).
>
> Francesco
>
>
> _______________________________________________
> dev-l10n mailing list
>
dev-...@lists.mozilla.org
>
https://lists.mozilla.org/listinfo/dev-l10n
>