)]}'
{"tools/run_pep8.sh":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"ab8d7241f8d8629652d39048e64c712f6318f4a5","unresolved":false,"context_lines":[{"line_number":4,"context_line":"SCRIPT_ROOT\u003d$(echo $(dirname $(readlink -f \"$0\")) | sed s/\\\\/tools//)"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"srcfiles\u003d`find nova smoketests tools -type f -name \"*.py\" ! -wholename \"nova\\/openstack*\"`"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"# NOTE(lzyeval): Avoid selecting nova-api-paste.ini and nova.conf in nova/bin"},{"line_number":9,"context_line":"#                when running on devstack."},{"line_number":10,"context_line":"srcfiles+\u003d\" `find bin -type f ! -name \"nova.conf*\" ! -name \"*api-paste.ini*\" ! -name \"*~\"`\""}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAM3%2F%2B6FM%3D","line":7,"updated":"2013-02-14 20:04:57.000000000","message":"Why the extra blank lines here?","commit_id":"1f3e98d22636e555f3a674a0715eef7737634330"},{"author":{"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},"change_message_id":"85e45d6f1fcb9b0ce6e702e45fb5844d8ed20317","unresolved":false,"context_lines":[{"line_number":4,"context_line":"SCRIPT_ROOT\u003d$(echo $(dirname $(readlink -f \"$0\")) | sed s/\\\\/tools//)"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"srcfiles\u003d`find nova smoketests tools -type f -name \"*.py\" ! -wholename \"nova\\/openstack*\"`"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"# NOTE(lzyeval): Avoid selecting nova-api-paste.ini and nova.conf in nova/bin"},{"line_number":9,"context_line":"#                when running on devstack."},{"line_number":10,"context_line":"srcfiles+\u003d\" `find bin -type f ! -name \"nova.conf*\" ! -name \"*api-paste.ini*\" ! -name \"*~\"`\""}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAM3%2F%2B6Co%3D","line":7,"in_reply_to":"AAAAM3%2F%2B6FM%3D","updated":"2013-02-14 20:36:28.000000000","message":"Was looking a bit dense, but don\u0027t mind axing it.","commit_id":"1f3e98d22636e555f3a674a0715eef7737634330"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"ab8d7241f8d8629652d39048e64c712f6318f4a5","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"# NOTE(lzyeval): Avoid selecting nova-api-paste.ini and nova.conf in nova/bin"},{"line_number":9,"context_line":"#                when running on devstack."},{"line_number":10,"context_line":"srcfiles+\u003d\" `find bin -type f ! -name \"nova.conf*\" ! -name \"*api-paste.ini*\" ! -name \"*~\"`\""},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"srcfiles+\u003d\" setup.py\""},{"line_number":13,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAM3%2F%2B6Ew%3D","line":10,"updated":"2013-02-14 20:04:57.000000000","message":"The logic in run_tests.py and tox.ini were slightly different.  Since the tox.ini version is what gating uses, it would be nice to use that version here.","commit_id":"1f3e98d22636e555f3a674a0715eef7737634330"},{"author":{"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},"change_message_id":"85e45d6f1fcb9b0ce6e702e45fb5844d8ed20317","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"# NOTE(lzyeval): Avoid selecting nova-api-paste.ini and nova.conf in nova/bin"},{"line_number":9,"context_line":"#                when running on devstack."},{"line_number":10,"context_line":"srcfiles+\u003d\" `find bin -type f ! -name \"nova.conf*\" ! -name \"*api-paste.ini*\" ! -name \"*~\"`\""},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"srcfiles+\u003d\" setup.py\""},{"line_number":13,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAM3%2F%2B6Cc%3D","line":10,"in_reply_to":"AAAAM3%2F%2B6Ew%3D","updated":"2013-02-14 20:36:28.000000000","message":"Went in favor of run_tests because it was commented and so could understand why it was the way it was.\n\nThat said, don\u0027t mind using the tox version.","commit_id":"1f3e98d22636e555f3a674a0715eef7737634330"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"ab8d7241f8d8629652d39048e64c712f6318f4a5","unresolved":false,"context_lines":[{"line_number":22,"context_line":"# NOTE(sirp): Also check Dom0 plugins w/o .py extension"},{"line_number":23,"context_line":"PYTHONPATH\u003d$SCRIPT_ROOT/plugins/xenserver/xenapi/etc/xapi.d/plugins python tools/hacking.py ${ignore} ./plugins/xenserver/xenapi `find plugins/xenserver/xenapi/etc/xapi.d/plugins/ -type f -perm +111`"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"bash tools/unused_imports.sh"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAM3%2F%2B6FY%3D","line":25,"updated":"2013-02-14 20:04:57.000000000","message":"Can we remove the unused_imports script and move the one line over here","commit_id":"1f3e98d22636e555f3a674a0715eef7737634330"},{"author":{"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},"change_message_id":"85e45d6f1fcb9b0ce6e702e45fb5844d8ed20317","unresolved":false,"context_lines":[{"line_number":22,"context_line":"# NOTE(sirp): Also check Dom0 plugins w/o .py extension"},{"line_number":23,"context_line":"PYTHONPATH\u003d$SCRIPT_ROOT/plugins/xenserver/xenapi/etc/xapi.d/plugins python tools/hacking.py ${ignore} ./plugins/xenserver/xenapi `find plugins/xenserver/xenapi/etc/xapi.d/plugins/ -type f -perm +111`"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"bash tools/unused_imports.sh"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAM3%2F%2B6CY%3D","line":25,"in_reply_to":"AAAAM3%2F%2B6FY%3D","updated":"2013-02-14 20:36:28.000000000","message":"Good idea!","commit_id":"1f3e98d22636e555f3a674a0715eef7737634330"}]}