)]}'
{"contrib/redhat-eventlet.patch":[{"author":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"change_message_id":"e6b36afe9ae51fd6faaf4511aa047a46f72097b5","unresolved":false,"context_lines":[{"line_number":13,"context_line":"         # Instead of a blocking OS call, this version of wait() uses logic"},{"line_number":14,"context_line":"         # borrowed from the eventlet 0.2 processes.Process.wait() method."},{"line_number":15,"context_line":"         try:"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-diff","patch_set":3,"id":"AAAAN3%2F%2F2sQ%3D","line":16,"updated":"2013-03-08 19:11:29.000000000","message":"sorry, made this comment last review but I think you may have missed it as it was not an inline comment.  \n\nthe name of this patch implies that it is redhat specific, but  I don\u0027t see logic here that applies it only if the platform is redhat.  Can someone clarify this for me?  Thanks.","commit_id":"33c3cc559cddc9a4f1c5be502e0f83de140cad44"}],"tools/patch_tox_venv.py":[{"author":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"change_message_id":"e6b36afe9ae51fd6faaf4511aa047a46f72097b5","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    pip_requires \u003d os.path.join(root, \u0027tools\u0027, \u0027pip-requires\u0027)"},{"line_number":29,"context_line":"    test_requires \u003d os.path.join(root, \u0027tools\u0027, \u0027test-requires\u0027)"},{"line_number":30,"context_line":"    py_version \u003d \"python%s.%s\" % (sys.version_info[0], sys.version_info[1])"},{"line_number":31,"context_line":"    project \u003d \u0027Nova\u0027"},{"line_number":32,"context_line":"    install \u003d install_venv.InstallVenv(root, venv, pip_requires, test_requires,"},{"line_number":33,"context_line":"                                       py_version, project)"},{"line_number":34,"context_line":"    #NOTE(dprince): For Tox we only run post_process (which patches files, etc)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAN3%2F%2F3x8%3D","line":31,"updated":"2013-03-08 02:24:20.000000000","message":"Odd that it is not quantum.  is this just ignored?","commit_id":"e8211325f00f2f2fdde69a008ba9b6fc67ae8e30"},{"author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"unresolved":false,"context_lines":[{"line_number":28,"context_line":"    pip_requires \u003d os.path.join(root, \u0027tools\u0027, \u0027pip-requires\u0027)"},{"line_number":29,"context_line":"    test_requires \u003d os.path.join(root, \u0027tools\u0027, \u0027test-requires\u0027)"},{"line_number":30,"context_line":"    py_version \u003d \"python%s.%s\" % (sys.version_info[0], sys.version_info[1])"},{"line_number":31,"context_line":"    project \u003d \u0027Nova\u0027"},{"line_number":32,"context_line":"    install \u003d install_venv.InstallVenv(root, venv, pip_requires, test_requires,"},{"line_number":33,"context_line":"                                       py_version, project)"},{"line_number":34,"context_line":"    #NOTE(dprince): For Tox we only run post_process (which patches files, etc)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAN3%2F%2F3Ms%3D","line":31,"in_reply_to":"AAAAN3%2F%2F3x8%3D","updated":"2013-03-08 12:34:13.000000000","message":"Oops. I vetted this idea in Nova first and shamelessly copied it here. For the purposes of patching tox it is ignored... probably best to swap it out for Quantum though.","commit_id":"e8211325f00f2f2fdde69a008ba9b6fc67ae8e30"}]}
