602105 - [ro] backing out 6d4a9f32e233 to get builds going again

Status: RESOLVED FIXED

(Mozilla Localizations :: ro / Romanian, defect)

Description

[Axel Hecht [:Pike]]

http://hg.mozilla.org/l10n-central/ro/rev/6d4a9f32e233 flipped on some bad performing regular expressions in compare-locales, AFAICT.

I haven't found a way to solve this on my side, and even if I did, I'd probably won't be able to quickly get it deployed in our build systems, which currently kill l10n-merge mid-run.

I've backed this changeset out for now.

I suspect this is fall-out from trying to not export unchanged strings from narro, which is cool in itself, but it leaves a very long and winding series of comments at the end of http://hg.mozilla.org/l10n-central/ro/diff/6d4a9f32e233/toolkit/chrome/global/headsUpDisplay.properties, which is the actual file that's caughing up.

Alexandru, can you test new exports on both compare-locales 0.8.3 and 0.9 and report problems before landing them? That'd be much better than me taking the dashboard down for 2 hours and your builds being broken :-/

Comment 1

[Axel Hecht [:Pike]]

Landed http://hg.mozilla.org/l10n-central/ro/rev/b9d90a08e2c2, both dashboard and tinderbox builds work again.

Leaving this open for investigation.

Comment 2

[Axel Hecht [:Pike]]

Resolving this FIXED.

I've got a note in my head to check the parsers for ugly scaling regexps.