)]}'
{"bin/keystone-all":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"cc424d99e17960449974b8490f647c61c81aa69e","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from paste import deploy"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from keystone.openstack.common import gettextutils"},{"line_number":25,"context_line":"gettextutils.install(\u0027keystone\u0027)"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"from keystone import config"},{"line_number":28,"context_line":"from keystone.common import wsgi"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAOn%2F%2FeI4%3D","line":25,"updated":"2013-05-08 17:52:53.000000000","message":"would like a comment here saying this must be done before any other keystone imports.","commit_id":"3fe820ceaa38c77fa5ea8d68c3d5257d3ca2c0a3"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"d021033f2c8ca82a39a311f416c529bbdc1c6e06","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from paste import deploy"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from keystone.openstack.common import gettextutils"},{"line_number":25,"context_line":"gettextutils.install(\u0027keystone\u0027)"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"from keystone import config"},{"line_number":28,"context_line":"from keystone.common import wsgi"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAPH%2F%2F44g%3D","line":25,"in_reply_to":"AAAAOn%2F%2FeI4%3D","updated":"2013-05-20 22:05:00.000000000","message":"Looking at this again, it appears keystone doesn\u0027t do any string translations at module import time ... so I\u0027ve moved this","commit_id":"3fe820ceaa38c77fa5ea8d68c3d5257d3ca2c0a3"}],"bin/keystone-manage":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"cc424d99e17960449974b8490f647c61c81aa69e","unresolved":false,"context_lines":[{"line_number":14,"context_line":"    sys.path.insert(0, possible_topdir)"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from keystone.openstack.common import gettextutils"},{"line_number":17,"context_line":"gettextutils.install(\u0027keystone\u0027)"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from keystone import cli"},{"line_number":20,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAOn%2F%2FeI0%3D","line":17,"updated":"2013-05-08 17:52:53.000000000","message":"would like a comment here saying this must be done before any other keystone imports.","commit_id":"3fe820ceaa38c77fa5ea8d68c3d5257d3ca2c0a3"}],"keystone/openstack/common/gettextutils.py":[{"author":{"_account_id":7473,"name":"Grant Murphy","email":"grantcmurphy@gmail.com","username":"gm"},"change_message_id":"fa493f3ed725c6ce122ceb36158a9b352d18553a","unresolved":false,"context_lines":[{"line_number":26,"context_line":"import gettext"},{"line_number":27,"context_line":"import os"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"_localedir \u003d os.environ.get(\u0027keystone\u0027.upper() + \u0027_LOCALEDIR\u0027)"},{"line_number":30,"context_line":"_t \u003d gettext.translation(\u0027keystone\u0027, localedir\u003d_localedir, fallback\u003dTrue)"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOn%2F%2FdN8%3D","line":29,"updated":"2013-05-09 00:32:02.000000000","message":"This is being a bit picky but is the call to .upper() and string concatenation necessary here?","commit_id":"3fe820ceaa38c77fa5ea8d68c3d5257d3ca2c0a3"},{"author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"change_message_id":"2535079f50814761d2c7b416f9ab211bec08e720","unresolved":false,"context_lines":[{"line_number":26,"context_line":"import gettext"},{"line_number":27,"context_line":"import os"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"_localedir \u003d os.environ.get(\u0027keystone\u0027.upper() + \u0027_LOCALEDIR\u0027)"},{"line_number":30,"context_line":"_t \u003d gettext.translation(\u0027keystone\u0027, localedir\u003d_localedir, fallback\u003dTrue)"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOn%2F%2FZxg%3D","line":29,"in_reply_to":"AAAAOn%2F%2FdN8%3D","updated":"2013-05-10 18:49:45.000000000","message":"Done","commit_id":"3fe820ceaa38c77fa5ea8d68c3d5257d3ca2c0a3"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"d021033f2c8ca82a39a311f416c529bbdc1c6e06","unresolved":false,"context_lines":[{"line_number":26,"context_line":"import gettext"},{"line_number":27,"context_line":"import os"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"_localedir \u003d os.environ.get(\u0027keystone\u0027.upper() + \u0027_LOCALEDIR\u0027)"},{"line_number":30,"context_line":"_t \u003d gettext.translation(\u0027keystone\u0027, localedir\u003d_localedir, fallback\u003dTrue)"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2F45o%3D","line":29,"in_reply_to":"AAAAOn%2F%2FdN8%3D","updated":"2013-05-20 22:05:00.000000000","message":"It\u0027s this way because oslo\u0027s update.py changes \u0027oslo\u0027 to \u0027keystone\u0027","commit_id":"3fe820ceaa38c77fa5ea8d68c3d5257d3ca2c0a3"},{"author":{"_account_id":7138,"name":"David Jia","email":"jiataotj@cn.ibm.com","username":"jiataotj"},"change_message_id":"6925b7244d9d3a01c68c1539383e96ba6cba53d8","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    NOVA_LOCALEDIR)."},{"line_number":47,"context_line":"    \"\"\""},{"line_number":48,"context_line":"    gettext.install(domain,"},{"line_number":49,"context_line":"                    localedir\u003dos.environ.get(domain.upper() + \u0027_LOCALEDIR\u0027),"},{"line_number":50,"context_line":"                    unicode\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2Fpss%3D","line":49,"updated":"2013-05-28 15:28:29.000000000","message":"If we want to use default localedir /usr/share/locale, then, we need manually configure it as KEYSTONE_LOCALDIR value since we\u0027ll use this new install fucntion. right? If yes, I think we may should default use /usr/share/locale when KEYSTONE_LOCALDIR isn\u0027t configured","commit_id":"48768b0033bea6ccff82e5002069f671bfbe6664"}],"keystone/test.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"cc424d99e17960449974b8490f647c61c81aa69e","unresolved":false,"context_lines":[{"line_number":48,"context_line":"eventlet.patcher.monkey_patch(all\u003dFalse, socket\u003dTrue, time\u003dTrue,"},{"line_number":49,"context_line":"                              thread\u003ddo_monkeypatch)"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"gettext.install(\u0027keystone\u0027, unicode\u003d1)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":54,"context_line":"ROOTDIR \u003d os.path.dirname(os.path.abspath(os.curdir))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOn%2F%2FeII%3D","line":51,"updated":"2013-05-08 17:52:53.000000000","message":"isn\u0027t this supposed to be done before all the keystone imports? (like at line 33)","commit_id":"3fe820ceaa38c77fa5ea8d68c3d5257d3ca2c0a3"}]}