babel 0.9.6 can't compile tl_PH locale

Bug #1233694 reported by Matt Riedemann on 2013-10-01
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Undecided
Jay Bryant

Bug Description

There is a new tl_PH locale in cinder as of 9/28:

https://github.com/openstack/cinder/commits/master/cinder/locale/tl_PH/LC_MESSAGES

Using babel to compile the message catalog with babel 0.9.6 fails for that locale:

http://paste.openstack.org/show/47762/

bash-4.1$ pip show babel
---
Name: Babel
Version: 0.9.6
Location: /usr/lib/python2.6/site-packages
Requires:
bash-4.1$
bash-4.1$ pybabel --version
pybabel 0.9.6

The requirements on cinder for babel are >= 0.9.6:

https://github.com/openstack/cinder/blob/master/requirements.txt#L5

Looking at the po file, it's generated using babel 1.3:

https://github.com/openstack/cinder/blob/master/cinder/locale/tl_PH/LC_MESSAGES/cinder.po#L19

So either the translations need to be done with babel 0.9.6 or the requirements in the projects need to be updated to babel 1.3.

Matt Riedemann (mriedem) wrote :

Cinder will need to update to babel 1.1 after this is merged:

https://review.openstack.org/#/c/48739/

Matt Riedemann (mriedem) wrote :

Actually looks like zh_CN doesn't work with babel 1.1, so really we should be updating to 1.3. I get this when compiling against babel 1.1:

babel.core.UnknownLocaleError: unknown locale 'zh_CN'

The release notes for babel also point out a fix for zh_CN in 1.3:

http://babel.pocoo.org/docs/changelog/#version-1-3

Jay Bryant (jsbryant) on 2013-10-01
Changed in cinder:
assignee: nobody → Jay Bryant (jsbryant)
Matt Riedemann (mriedem) wrote :

global-requirements is being updated to babel >= 1.3 here now: https://review.openstack.org/#/c/48739/

Changed in cinder:
status: New → In Progress

Reviewed: https://review.openstack.org/49161
Committed: http://github.com/openstack/cinder/commit/56d3392e2dd66710a0876a8ba4d567e5d89086fb
Submitter: Jenkins
Branch: master

commit 56d3392e2dd66710a0876a8ba4d567e5d89086fb
Author: John Griffith <email address hidden>
Date: Tue Oct 1 08:23:01 2013 -0600

    Update Babel from Openstack Requirements

    This is the update pushed to the global reqs.

    Change-Id: I0d8ea330db3f9d68fee172302543b861cae4e013
    Closes-Bug: 1233694

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2013-10-04
Changed in cinder:
milestone: none → havana-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in cinder:
milestone: havana-rc1 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers