)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"505793e60b67a9580bd7864cd2d220ed8dfa0e14","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Don\u0027t hardcode the path to pip"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In at least some circumstances, our pip is installed into"},{"line_number":10,"context_line":"/usr/local/bin to make sure that\u0027s added to the environment and"},{"line_number":11,"context_line":"switch to dynamic path resolution when calling pip."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4e53ce2aba1b8335025e180bd03a5f5e93461814"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df87a7cf_abe91ee3","line":10,"updated":"2017-12-12 17:30:18.000000000","message":"Should say \"so make sure\" but I won\u0027t bother changing it unless another reviewer feels strongly about it or if I end up needing a new patchset for some other unrelated reason.","commit_id":"e02cfe94a364b9aa880d026c8e9089fe4ffb2829"}],"manifests/init.pp":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"de948e4d906fe4f546263c89065657335cda7b9d","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"  exec { \u0027install-subunit2sql-safely\u0027:"},{"line_number":41,"context_line":"    command     \u003d\u003e \u0027pip install --upgrade --upgrade-strategy\u003donly-if-needed subunit2sql\u0027,"},{"line_number":42,"context_line":"    environment \u003d\u003e \u0027PATH\u003d/usr/local/bin:/usr/bin:/bin/\u0027,"},{"line_number":43,"context_line":"    # This checks the current installed subunit2sql version with pip list and"},{"line_number":44,"context_line":"    # the latest version of subunit2sql on pypi with pip search and if they are"},{"line_number":45,"context_line":"    # different then we know we need to upgrade to reconcile the local version"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"df87a7cf_ab2efe88","line":42,"updated":"2017-12-12 17:05:50.000000000","message":"The cult of cargo has failed me. Seems our validation says I should use a path parameter rather than environment here.","commit_id":"09c9cea5995d68d3025e4fdbcb00343f3188461b"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"49effd73a00aa33fe47d3796f27eb16d22e344cf","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"  exec { \u0027install-subunit2sql-safely\u0027:"},{"line_number":41,"context_line":"    command \u003d\u003e \u0027pip install --upgrade --upgrade-strategy\u003donly-if-needed subunit2sql\u0027,"},{"line_number":42,"context_line":"    path    \u003d\u003e \u0027/usr/local/bin:/usr/bin:/bin/\u0027,"},{"line_number":43,"context_line":"    # This checks the current installed subunit2sql version with pip list and"},{"line_number":44,"context_line":"    # the latest version of subunit2sql on pypi with pip search and if they are"},{"line_number":45,"context_line":"    # different then we know we need to upgrade to reconcile the local version"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"df87a7cf_010eea31","line":42,"updated":"2017-12-12 20:13:05.000000000","message":"i\u0027d think this was the path, but i don\u0027t suppose it matters.  only concern would be there\u0027s no sbins in there, but likely this doesn\u0027t need anything in there","commit_id":"e02cfe94a364b9aa880d026c8e9089fe4ffb2829"}]}