Closed Bug 662133 Opened 14 years ago Closed 14 years ago

Nounclass3 strings are required for cs locale

Categories

(Calendar :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: Fallen, Unassigned)

Details

(Whiteboard: [needed beta][has l10n impact])

Attachments

(1 file)

Attached patch Fix - v1 β€” β€” Splinter Review
The cs locale requires a third noun class. This is causing compare-locales to complain about "obsolete" strings, since the third noun class strings are not in en-US. See [1]. Decathlon, I hope I did everything correct here, you might want to double check that. If I missed something, feel free to put up a counter-patch. Also, Pike, how will this patch influence the other locales? Thanks to [2] we now filter out the noun class strings, so iiuc then even if we push this patch the other locales will not have to make any changes? [1] https://l10n-stage-sj.mozilla.org/dashboard/compare?run=138878 [2] http://hg.mozilla.org/comm-central/rev/c3d6867fa36f
Attachment #537429 - Flags: review?(bv1578)
Flags: blocking-calendar1.0+
In this case I've completely misunderstood the meaning of the bug 577695 and the related patch. I thought it had just to avoid to add strings Nounclass...x (x>2) in the en-US locale. Anyway a third noun class could be added because, if I remember correctly, there are more languages that have (the equivalent of) a "neutral" gender besides (the equivalent of) masculine and feminine genders for the week days.
Our filter.py was just incorrect, this patch shouldn't be needed!
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → INVALID
Attachment #537429 - Flags: review?(bv1578)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: