In http://mxr.mozilla.org/l10n/search?string=FileError&find=[beimru][deku]%2Fbrowser%2Finstaller&findi=&filter=^[^\0]*%24&hitlimit=&tree=l10n and http://mxr.mozilla.org/l10n/search?string=FileError&find=[ez][sh]-[EC].%2Fbrowser%2Finstaller&findi=&filter=^[^\0]*%24&hitlimit=&tree=l10n, the original markup of the newlines in the error messages got mangled and changed. I don't see this as a release blocker, as apparently preprocess-locale.pl does 'something', but the newline combo should really look like en-US, i.e., \r\n\r\n$0\r\n\r\n and \r\n elsewhere. This affects both FileError and FileError_NoIgnore. Please attach a patch and request approval1.9 to include this in a future 3.0.x release.
Created attachment 324184 [details] [diff] [review] the patch
Comment on attachment 324184 [details] [diff] [review] the patch a=me, please land this with a check-in comment referencing this bug and my approval. Something like "bug 324184, fix newlines, email@example.com" would work. Please use the fixed22.214.171.124 and verified126.96.36.199 keywords to track landing and testing. Please cancel the approval1.9 flag, too.
Attachment #324184 - Flags: approval188.8.131.52+
Comment on attachment 324184 [details] [diff] [review] the patch I can actually cancel myself ;-)
Landed. Checking in id/browser/installer/override.properties; /l10n/l10n/id/browser/installer/override.properties,v <-- override.properties new revision: 1.3; previous revision: 1.2 done Thanks Axel
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.