[be] installer/overrides.properties newline cleanup

RESOLVED FIXED

Status

Mozilla Localizations
be / Belarusian
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: Pike, Assigned: Siarhei)

Tracking

({fixed1.9.0.1, verified1.9.0.1})

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
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.
(Assignee)

Comment 1

10 years ago
Created attachment 325180 [details] [diff] [review]
override.properties patch
Attachment #325180 - Flags: approval1.9?
(Assignee)

Updated

10 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

10 years ago
Attachment #325180 - Flags: approval1.9? → approval1.9.0.1?
(Reporter)

Comment 2

10 years ago
Comment on attachment 325180 [details] [diff] [review]
override.properties patch

a=me, please land this with a check-in comment referencing this bug and my approval.

Please use the fixed1.9.0.1 and verified1.9.0.1 keywords to track landing and testing.
Attachment #325180 - Flags: approval1.9.0.1? → approval1.9.0.1+
(Assignee)

Updated

10 years ago
Keywords: fixed1.9.0.1
(Assignee)

Updated

10 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Keywords: verified1.9.0.1
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.