)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"bd0d1ae43879b5eaa9952a43245da4175ae5c3c8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Monty Taylor \u003cmordred@inaugust.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2012-02-20 11:57:13 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Split requires."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I4881fa8bfa2eedf55587c05ed264208452fda99e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAEn%2F%2F4gs%3D","line":7,"updated":"2012-02-21 13:37:14.000000000","message":"Might be an idea to expand the commit description here slightly, e.g.\n\nSplit requires used to build virtualenv into separate test and runtime dependency sets.\n\n(sub\u0027ing in appropriate terminology ...)","commit_id":"620deafdf7e153e3a8af9fcccd80bed70ecaa573"}],"tools/install_venv.py":[{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"c8444c8f14ff7f3b90d6472cd4e13dc8252b655f","unresolved":false,"context_lines":[{"line_number":30,"context_line":"ROOT \u003d os.path.dirname(os.path.dirname(os.path.realpath(__file__)))"},{"line_number":31,"context_line":"VENV \u003d os.path.join(ROOT, \u0027.venv\u0027)"},{"line_number":32,"context_line":"PIP_REQUIRES \u003d os.path.join(ROOT, \u0027tools\u0027, \u0027pip-requires\u0027)"},{"line_number":33,"context_line":"TEST_REQUIRES \u003d os.path.join(ROOT, \u0027tools\u0027, \u0027test-requires\u0027)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"def die(message, *args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAEn%2F%2F4Qk%3D","line":33,"updated":"2012-02-21 21:39:50.000000000","message":"Can you keep \u0027pip\u0027 in the name here? We\u0027re still using pip to install these deps, they\u0027re just focused on testing. Maybe \u0027pip-requires-test\u0027 like in keystone?","commit_id":"620deafdf7e153e3a8af9fcccd80bed70ecaa573"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"cefe5982026550eff77aad1e21cc35946b0b48f5","unresolved":false,"context_lines":[{"line_number":30,"context_line":"ROOT \u003d os.path.dirname(os.path.dirname(os.path.realpath(__file__)))"},{"line_number":31,"context_line":"VENV \u003d os.path.join(ROOT, \u0027.venv\u0027)"},{"line_number":32,"context_line":"PIP_REQUIRES \u003d os.path.join(ROOT, \u0027tools\u0027, \u0027pip-requires\u0027)"},{"line_number":33,"context_line":"TEST_REQUIRES \u003d os.path.join(ROOT, \u0027tools\u0027, \u0027test-requires\u0027)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"def die(message, *args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAEn%2F%2F16I%3D","line":33,"in_reply_to":"AAAAEn%2F%2F1%2Bs%3D","updated":"2012-02-26 21:29:29.000000000","message":"\u0027other places\u0027 as in other projects? If so, then I\u0027ll let it go. But mark my words, Monty Taylor, I will get my way.","commit_id":"620deafdf7e153e3a8af9fcccd80bed70ecaa573"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"e5802ae981d2ac2299966b14302831666f0588f8","unresolved":false,"context_lines":[{"line_number":30,"context_line":"ROOT \u003d os.path.dirname(os.path.dirname(os.path.realpath(__file__)))"},{"line_number":31,"context_line":"VENV \u003d os.path.join(ROOT, \u0027.venv\u0027)"},{"line_number":32,"context_line":"PIP_REQUIRES \u003d os.path.join(ROOT, \u0027tools\u0027, \u0027pip-requires\u0027)"},{"line_number":33,"context_line":"TEST_REQUIRES \u003d os.path.join(ROOT, \u0027tools\u0027, \u0027test-requires\u0027)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"def die(message, *args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAEn%2F%2F1%2Bs%3D","line":33,"in_reply_to":"AAAAEn%2F%2F4Qk%3D","updated":"2012-02-26 01:44:16.000000000","message":"Well, they\u0027re already TEST_REQUIRES lots of other places. I could make them pip-requires-test... although I would like to make pip-requires-test in new keystone look like this one (additive rather than alternate)\n\nThoughts?","commit_id":"620deafdf7e153e3a8af9fcccd80bed70ecaa573"}]}