bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

change .po compile scripts to use --check-header

RESOLVED FIXED

Status

Input
General
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: willkg, Assigned: willkg)

Tracking

Details

Sometimes something happens to the Plurals: header in the .po file. If this isn't valid, then all kinds of bad things happen.

This has happened to SUMO. After it happened, I decided I'd add some Plurals linting to Dennis. I never did that work, but I still think about it from time to time usually after seeing a shooting star.

However, we don't need to do that. msgfmt has a --check-header flag which does this (and more) and errors out if the Plurals is malformed.

Given that, we should update our compile scripts and add the --check-header flag.
Grabbing this now.
Assignee: nobody → willkg
Status: NEW → ASSIGNED
In a PR: https://github.com/mozilla/fjord/pull/696

Also, the header is named "Plural-Forms"--not "Plurals".
Landed in https://github.com/mozilla/fjord/commit/f48d9b872acfd2535862a02ccdf1118d84c378cc

Pushed to -stage and -prod just now.
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.