)]}'
{"doc/source/_extra/.htaccess":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"715081b7b195c8671c0a85d2753b813c1225299d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9f5c4f37_8432d7a6","updated":"2017-09-21 07:03:44.000000000","message":"Why do we need this if its empty? Let\u0027s remove it - it will just slow down our web server since it will be read everytime...","commit_id":"c323784ef3d033663b4c298fd8195033d4814ba9"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"98754ccbc52aff291a924d83e81d60c9d0acd4bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9f5c4f37_8d09d3de","in_reply_to":"9f5c4f37_8432d7a6","updated":"2017-09-21 12:22:08.000000000","message":"Because the job looks for whereto to be installed but doesn\u0027t check that the input files exist before it tries to run whereto.","commit_id":"c323784ef3d033663b4c298fd8195033d4814ba9"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"2f3774378ce083825092daa38e6b0f47054290b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9f5c4f37_b6edff26","in_reply_to":"9f5c4f37_8d09d3de","updated":"2017-09-21 15:41:10.000000000","message":"That sounds like a bug with whereto, no?","commit_id":"c323784ef3d033663b4c298fd8195033d4814ba9"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"d6e7f05de562c6af0c81e6c162b925be5869085e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9f5c4f37_87dc0747","in_reply_to":"9f5c4f37_b6edff26","updated":"2017-09-21 17:26:41.000000000","message":"I would expect whereto to complain if the input files it is given don\u0027t exist. I could change the job to check for whereto and both input files, I guess, but it seems like in most cases including whereto but not having the files in the right places is an error in configuration. It\u0027s a special case for the whereto repository, because whereto is always going to be installed, since it\u0027s the application being tested.","commit_id":"c323784ef3d033663b4c298fd8195033d4814ba9"}],"tox.ini":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"e9087ecb2a3267f4789d174424138dc30b3ddfe7","unresolved":false,"context_lines":[{"line_number":8,"context_line":"install_command \u003d {toxinidir}/tools/tox_install.sh {env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages}"},{"line_number":9,"context_line":"setenv \u003d"},{"line_number":10,"context_line":"    VIRTUAL_ENV\u003d{envdir}"},{"line_number":11,"context_line":"    BRANCH_NAME\u003dmaster"},{"line_number":12,"context_line":"    CLIENT_NAME\u003dwhereto"},{"line_number":13,"context_line":"    PYTHONWARNINGS\u003ddefault::DeprecationWarning"},{"line_number":14,"context_line":"deps \u003d -r{toxinidir}/test-requirements.txt"}],"source_content_type":"text/x-properties","patch_set":1,"id":"bf57530f_301eb30c","line":11,"range":{"start_line":11,"start_character":3,"end_line":11,"end_character":22},"updated":"2017-09-20 16:17:04.000000000","message":"I don\u0027t see this used anywhere?","commit_id":"c5d7a0e3e6c6b252dbde7023f5bb00717df190ef"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"0e5267c9eef7c748d8c99a38da3586e1f3c3abd5","unresolved":false,"context_lines":[{"line_number":8,"context_line":"install_command \u003d {toxinidir}/tools/tox_install.sh {env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages}"},{"line_number":9,"context_line":"setenv \u003d"},{"line_number":10,"context_line":"    VIRTUAL_ENV\u003d{envdir}"},{"line_number":11,"context_line":"    BRANCH_NAME\u003dmaster"},{"line_number":12,"context_line":"    CLIENT_NAME\u003dwhereto"},{"line_number":13,"context_line":"    PYTHONWARNINGS\u003ddefault::DeprecationWarning"},{"line_number":14,"context_line":"deps \u003d -r{toxinidir}/test-requirements.txt"}],"source_content_type":"text/x-properties","patch_set":1,"id":"bf57530f_5055a74f","line":11,"range":{"start_line":11,"start_character":3,"end_line":11,"end_character":22},"in_reply_to":"bf57530f_301eb30c","updated":"2017-09-20 16:20:16.000000000","message":"Weird. I copy-pasta-ed that from another patch that used this script, but you\u0027re right. I\u0027ll remove it in a follow-up patch.","commit_id":"c5d7a0e3e6c6b252dbde7023f5bb00717df190ef"}]}
