)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"1c33b2a9c27ae5c6c87bd080f4f4dcb618904d6a","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Perform minimal checks on po and pot files so that no broken files can"},{"line_number":10,"context_line":"get imported."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Mention msgfmt requirement."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ifa9fe2fc1ca9a41621ff3039c249700cdb1cd0ac"},{"line_number":15,"context_line":"Related-Bug: #1299349"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAAXH%2F%2F3yA%3D","line":12,"updated":"2014-04-16 19:26:16.000000000","message":"You should probably say something about adding msgfmt to list of requirements in the setup doc.","commit_id":"43796fa5ac55820a64f8622f7d7645ee91c659d4"}],"doc/source/setup.rst":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"1c33b2a9c27ae5c6c87bd080f4f4dcb618904d6a","unresolved":false,"context_lines":[{"line_number":42,"context_line":"- git_"},{"line_number":43,"context_line":"- setuptools_"},{"line_number":44,"context_line":"- pip_"},{"line_number":45,"context_line":"- msgfmt_ (part of the gettext package)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"**Reminder**: If you\u0027re successfully using a different platform, or a"},{"line_number":48,"context_line":"different version of the above, please document your configuration here!"}],"source_content_type":"text/x-rst","patch_set":7,"id":"AAAAXH%2F%2F3xQ%3D","line":45,"updated":"2014-04-16 19:26:16.000000000","message":"Having the trailing underscore means that you\u0027ll be providing a link. See lines 50 and 51. If you don\u0027t actually want a link, just remove the underscore.","commit_id":"43796fa5ac55820a64f8622f7d7645ee91c659d4"}],"tox.ini":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"619893e14625f7835cfe26a8be1f687a124e48b5","unresolved":false,"context_lines":[{"line_number":17,"context_line":"commands \u003d "},{"line_number":18,"context_line":"  flake8 {posargs}"},{"line_number":19,"context_line":"  # Check that .po and .pot files are valid:"},{"line_number":20,"context_line":"  bash -c \"find keystone -type f -regex \u0027.*\\.pot?\u0027 -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[tox:jenkins]"},{"line_number":23,"context_line":"downloadcache \u003d ~/cache/pip"}],"source_content_type":"text/x-properties","patch_set":5,"id":"AAAAW3%2F%2FiJE%3D","line":20,"updated":"2014-04-13 15:43:25.000000000","message":"suggest use find -exec, then don\u0027t need to bash.","commit_id":"770881b5003e1bb2419a4046a65c3f3f0b31af17"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"6d77d83fff423f04f84cdf4df9125b9366f3efd7","unresolved":false,"context_lines":[{"line_number":17,"context_line":"commands \u003d "},{"line_number":18,"context_line":"  flake8 {posargs}"},{"line_number":19,"context_line":"  # Check that .po and .pot files are valid:"},{"line_number":20,"context_line":"  bash -c \"find keystone -type f -regex \u0027.*\\.pot?\u0027 -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[tox:jenkins]"},{"line_number":23,"context_line":"downloadcache \u003d ~/cache/pip"}],"source_content_type":"text/x-properties","patch_set":5,"id":"AAAAW3%2F%2FiAI%3D","line":20,"in_reply_to":"AAAAW3%2F%2FiJE%3D","updated":"2014-04-13 16:45:42.000000000","message":"That was my first iteration - but it does not work - the call will always sucee even if the msgfmt fails. And we want the call to failed.\n\nTestcase on my Linux system, \"find -exec false {} \\;\" will return 0 while \"false\" will return \"1\".","commit_id":"770881b5003e1bb2419a4046a65c3f3f0b31af17"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"ffc17f03bbb3f9ba2cc9584603540c6f8d5a8b2a","unresolved":false,"context_lines":[{"line_number":17,"context_line":"commands \u003d "},{"line_number":18,"context_line":"  flake8 {posargs}"},{"line_number":19,"context_line":"  # Check that .po and .pot files are valid:"},{"line_number":20,"context_line":"  bash -c \"find keystone -type f -regex \u0027.*\\.pot?\u0027 -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[tox:jenkins]"},{"line_number":23,"context_line":"downloadcache \u003d ~/cache/pip"}],"source_content_type":"text/x-properties","patch_set":8,"id":"AAAAXH%2F%2FAnc%3D","line":20,"updated":"2014-04-24 17:34:23.000000000","message":"I assume the first part of this is intended to match all the .po and .pot files? it doesn\u0027t match anything for me... is it supposed to be an asterisk instead of a question mark?\n\n  find keystone -type f -regex \u0027.*\\.pot*\u0027","commit_id":"463903557765e67250ce675ab81829d6f49066c9"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"ff2ff8a08fff7c75657717a640ee7af98751c0de","unresolved":false,"context_lines":[{"line_number":17,"context_line":"commands \u003d "},{"line_number":18,"context_line":"  flake8 {posargs}"},{"line_number":19,"context_line":"  # Check that .po and .pot files are valid:"},{"line_number":20,"context_line":"  bash -c \"find keystone -type f -regex \u0027.*\\.pot?\u0027 -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[tox:jenkins]"},{"line_number":23,"context_line":"downloadcache \u003d ~/cache/pip"}],"source_content_type":"text/x-properties","patch_set":8,"id":"AAAAXX%2F%2F27c%3D","line":20,"in_reply_to":"AAAAXH%2F%2FAJQ%3D","updated":"2014-04-25 18:14:15.000000000","message":"whatever is included with OS X 10.9 Mavericks (i\u0027d check for you, but I\"m not on that machine at the moment)","commit_id":"463903557765e67250ce675ab81829d6f49066c9"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"e4c988c10005bb9241014adb5578e74cf9883a29","unresolved":false,"context_lines":[{"line_number":17,"context_line":"commands \u003d "},{"line_number":18,"context_line":"  flake8 {posargs}"},{"line_number":19,"context_line":"  # Check that .po and .pot files are valid:"},{"line_number":20,"context_line":"  bash -c \"find keystone -type f -regex \u0027.*\\.pot?\u0027 -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[tox:jenkins]"},{"line_number":23,"context_line":"downloadcache \u003d ~/cache/pip"}],"source_content_type":"text/x-properties","patch_set":8,"id":"AAAAXH%2F%2FAJQ%3D","line":20,"in_reply_to":"AAAAXH%2F%2FAnc%3D","updated":"2014-04-24 18:25:52.000000000","message":"Strange, this works for me:\n$ find keystone -type f -regex \u0027.*\\.pot?\u0027 \nkeystone/locale/ka_GE/LC_MESSAGES/keystone.po\nkeystone/locale/zh_HK/LC_MESSAGES/keystone.po\nkeystone/locale/tl_PH/LC_MESSAGES/keystone.po\n....\nYes, it should match all po and pot files - the \"t?\" means optionally a \"t\".\n\nWhat kind of find is this? I can change this to an asterisk if you prefer.","commit_id":"463903557765e67250ce675ab81829d6f49066c9"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"0d6593440feea47c5017cc0bf94f72c5b5e8a1ff","unresolved":false,"context_lines":[{"line_number":17,"context_line":"commands \u003d "},{"line_number":18,"context_line":"  flake8 {posargs}"},{"line_number":19,"context_line":"  # Check that .po and .pot files are valid:"},{"line_number":20,"context_line":"  bash -c \"find keystone -type f -regex \u0027.*\\.pot?\u0027 -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[tox:jenkins]"},{"line_number":23,"context_line":"downloadcache \u003d ~/cache/pip"}],"source_content_type":"text/x-properties","patch_set":8,"id":"AAAAXX%2F%2F2gw%3D","line":20,"in_reply_to":"AAAAXX%2F%2F27c%3D","updated":"2014-04-25 19:19:21.000000000","message":"On OS X 10.9, find requires the -E argument to properly process full regexes (BRE/POSIX.2) after the -regex argument. Linux has no such option.\n\nIt would be better to use the .*\\.pot* regex if we want to support this operation on OS X","commit_id":"463903557765e67250ce675ab81829d6f49066c9"}]}