babel 0.9.6 can't compile tl_PH locale

Bug #1233694 reported by Matt Riedemann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
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.

Revision history for this message
Matt Riedemann (mriedem) wrote :

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

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

Revision history for this message
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)
Changed in cinder:
assignee: nobody → Jay Bryant (jsbryant)
Revision history for this message
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
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

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)
Changed in cinder:
milestone: none → havana-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: havana-rc1 → 2013.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.