)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"ba39b2cd189cb3591e34db1b6d615102e83481f0","unresolved":false,"context_lines":[{"line_number":19,"context_line":"Note this is not as much of an issue on Debian/Ubuntu, as they keep"},{"line_number":20,"context_line":"their pip packages in a separate place to the system packages, so you"},{"line_number":21,"context_line":"don\u0027t have these overwite conflicts as much. Change-Id:"},{"line_number":22,"context_line":"Ib40708c07b939b84661c44df88a5a308fd0c7216"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Change-Id: Ib40708c07b939b84661c44df88a5a308fd0c7216"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7aa08908_0f11cc5d","line":22,"updated":"2016-06-16 16:40:37.000000000","message":"Just fyi, Change-Id is in here twice for some reason.","commit_id":"fb38098b6e6a5ef8361d0b913658e178ca805cab"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e50ebacb14858f8689cb66b31a39b3d9d888693e","unresolved":false,"context_lines":[{"line_number":19,"context_line":"Note this is not as much of an issue on Debian/Ubuntu, as they keep"},{"line_number":20,"context_line":"their pip packages in a separate place to the system packages, so you"},{"line_number":21,"context_line":"don\u0027t have these overwite conflicts as much. Change-Id:"},{"line_number":22,"context_line":"Ib40708c07b939b84661c44df88a5a308fd0c7216"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Change-Id: Ib40708c07b939b84661c44df88a5a308fd0c7216"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7aa08908_9b0e908a","line":22,"in_reply_to":"7aa08908_0f11cc5d","updated":"2016-06-16 23:13:56.000000000","message":"merge failure on my behalf, removed","commit_id":"fb38098b6e6a5ef8361d0b913658e178ca805cab"}],"elements/pip-and-virtualenv/install.d/pip-and-virtualenv-source-install/01-install-pip":[{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"b82802c14eb1eb3d20d7e58b050507b4545aae79","unresolved":false,"context_lines":[{"line_number":16,"context_line":"    # on somebody does a \"yum install python-virtualenv\" and goes and"},{"line_number":17,"context_line":"    # overwrites the pip installed version with the packaged version,"},{"line_number":18,"context_line":"    # leading to all sorts of weird version issues."},{"line_number":19,"context_line":"    ${YUM} install -y python-virtualenv python-pip python-setuptools"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    # install pip; this overwrites packaged pip"},{"line_number":22,"context_line":"    python /tmp/get-pip.py"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7aa08908_adc37656","line":19,"updated":"2016-06-10 11:42:17.000000000","message":"seems like the problem might occur again on a yum update now that the packages are installed.\n\nwhat about adding them to exclude in yum.conf?","commit_id":"c73204445330876a63b62bb8d4ee5ed30fb210c1"},{"author":{"_account_id":12459,"name":"Stephanie Miller","email":"stephane@alum.mit.edu","username":"cinerama"},"change_message_id":"10ad809f7edf8c5afda1e33335464ea32d690e66","unresolved":false,"context_lines":[{"line_number":16,"context_line":"    # on somebody does a \"yum install python-virtualenv\" and goes and"},{"line_number":17,"context_line":"    # overwrites the pip installed version with the packaged version,"},{"line_number":18,"context_line":"    # leading to all sorts of weird version issues."},{"line_number":19,"context_line":"    ${YUM} install -y python-virtualenv python-pip python-setuptools"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    # install pip; this overwrites packaged pip"},{"line_number":22,"context_line":"    python /tmp/get-pip.py"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7aa08908_adb7ddf3","line":19,"in_reply_to":"7aa08908_adc37656","updated":"2016-06-10 19:07:13.000000000","message":"We could do that but then when they attempt to install, the yum command will return an error which will potentially cause trouble with the other scripts.\n\nIt\u0027d be good to have a bug filed indicating the specific context(s) where this comes up so we can evaluate this solution more holistically.","commit_id":"c73204445330876a63b62bb8d4ee5ed30fb210c1"},{"author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"change_message_id":"f30f28e68bf2fbc9d3a67fc688617a6b7cfcaa6e","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    # requires setuptools for some operations, such as wheel creation."},{"line_number":32,"context_line":"    # But just installing setuptools shouldn\u0027t require setuptools"},{"line_number":33,"context_line":"    # itself, so we are safe for this small section."},{"line_number":34,"context_line":"    rm -rf /usr/lib/python2.7/site-packages/setuptools*"},{"line_number":35,"context_line":"    pip install -U setuptools"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    # now install latest virtualenv.  it vendors stuff it needs so"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7aa08908_86d79022","line":34,"updated":"2016-06-10 01:26:34.000000000","message":"What about on recent fedoras where python3 is the default - do we need to do something smarter about the correct path to rm?","commit_id":"c73204445330876a63b62bb8d4ee5ed30fb210c1"},{"author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"change_message_id":"7969d7d524b1e4495fe4cf4f099e572e57563124","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    # requires setuptools for some operations, such as wheel creation."},{"line_number":32,"context_line":"    # But just installing setuptools shouldn\u0027t require setuptools"},{"line_number":33,"context_line":"    # itself, so we are safe for this small section."},{"line_number":34,"context_line":"    rm -rf /usr/lib/python2.7/site-packages/setuptools*"},{"line_number":35,"context_line":"    pip install -U setuptools"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    # now install latest virtualenv.  it vendors stuff it needs so"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7aa08908_e60bf47b","line":34,"in_reply_to":"7aa08908_86d79022","updated":"2016-06-10 01:38:49.000000000","message":"nvm, chatted in irc and this should be fine - python-pip is for the python2 pip only so this isnt an issue.","commit_id":"c73204445330876a63b62bb8d4ee5ed30fb210c1"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"1798df56a0d1f3dcfa206beec3521c081372186b","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    # requires setuptools for some operations, such as wheel creation."},{"line_number":32,"context_line":"    # But just installing setuptools shouldn\u0027t require setuptools"},{"line_number":33,"context_line":"    # itself, so we are safe for this small section."},{"line_number":34,"context_line":"    rm -rf /usr/lib/python2.7/site-packages/setuptools*"},{"line_number":35,"context_line":"    pip install -U setuptools"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    # now install latest virtualenv.  it vendors stuff it needs so"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7aa08908_26021c3f","line":34,"in_reply_to":"7aa08908_86d79022","updated":"2016-06-10 01:43:20.000000000","message":"so firstly AFAICT building from the Fedora ISO has been broken for pretty much the whole of the F23 life-cycle (as opposed to fedora-minimal, which works fine).  So I don\u0027t think anyone is actually using that\n\nbut, we\u0027ve done the \"yum install\" of the python-packages above, which will have dragged in enough of python2 to be on the system anyway.\n\nsince python3 comes with pip, this will just basically leave it all alone.  there may very well be issues down that path, but until something starts using it (devstack, maybe) it is currently unexplored (see note about dragons :)","commit_id":"c73204445330876a63b62bb8d4ee5ed30fb210c1"},{"author":{"_account_id":12459,"name":"Stephanie Miller","email":"stephane@alum.mit.edu","username":"cinerama"},"change_message_id":"2b68c3c7f9226f6bdfd0a4a2a875ebb298710454","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    pip install -U virtualenv"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    # Add this to exclude so that we don\u0027t install a later package"},{"line_number":42,"context_line":"    # over it if it updates.  n.b. even dnf uses yum.conf (at least"},{"line_number":43,"context_line":"    # for now?)"},{"line_number":44,"context_line":"    if [[ -f /etc/yum.conf ]]; then"},{"line_number":45,"context_line":"        conf\u003d/etc/yum.conf"},{"line_number":46,"context_line":"    elif [[ -f /etc/dnf/dnf.conf ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"7aa08908_8afbea46","line":43,"range":{"start_line":42,"start_character":33,"end_line":43,"end_character":15},"updated":"2016-06-16 16:52:27.000000000","message":"I don\u0027t have a yum.conf on Fedora 23 but do have the dnf file, FYI.","commit_id":"fb38098b6e6a5ef8361d0b913658e178ca805cab"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e50ebacb14858f8689cb66b31a39b3d9d888693e","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    pip install -U virtualenv"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    # Add this to exclude so that we don\u0027t install a later package"},{"line_number":42,"context_line":"    # over it if it updates.  n.b. even dnf uses yum.conf (at least"},{"line_number":43,"context_line":"    # for now?)"},{"line_number":44,"context_line":"    if [[ -f /etc/yum.conf ]]; then"},{"line_number":45,"context_line":"        conf\u003d/etc/yum.conf"},{"line_number":46,"context_line":"    elif [[ -f /etc/dnf/dnf.conf ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"7aa08908_c7831da8","line":43,"range":{"start_line":42,"start_character":33,"end_line":43,"end_character":15},"in_reply_to":"7aa08908_8afbea46","updated":"2016-06-16 23:13:56.000000000","message":"yeah, it\u0027s confusing because the \"-minmal\" fedora builds are bootstrapped with yum and get it left behind, but /etc/dnf/dnf.conf is the one to edit.  i removed this","commit_id":"fb38098b6e6a5ef8361d0b913658e178ca805cab"}]}
