861168 - regression: l10n_merge no longer works, creates 0 bytes files

Status: RESOLVED FIXED

(www.mozilla.org :: Bedrock, defect)

Description

[Pascal Chevrel:pascalc]

1/ manage.py l10n_extract
2/ manage.py l10n_merge de

All the files created/updated by the call to l10n_merge are empty files.

Comment 1

[Jennifer Bertsch [:jbertsch]]

Hi Pmac-

Pascal asked if you could check on this one.  He's not sure if it is a regression or the settings on his new machine.

Thanks,
Jen

Comment 2

[Jennifer Bertsch [:jbertsch]]

Hi Pascal - Is this one still an issue for you?  Thx.

Comment 3

[Paul [:pmac] McLanahan]

I apparently forgot to update the bug. Sorry about that. I have a fix for this, but it's not yet merged. I gave pascalc my branch with the fix when he asked about it in IRC, but I need to write a test before I merge it. I'll try to get it in today.

Comment 4

[[github robot]]

Commit pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/8bcca89ccff01a9a6cad26875add4183fc9bd86e
Bug 861168: Fix regression in l10n_merge command.

Comment 5

[Paul [:pmac] McLanahan]

:pascalc: please check this out and let us know if it's working as you expect. Current master branch should be fixed.