)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"8eb682fa29f481f7af143df8c83a062b3f558991","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Also it doesn\u0027t match what will happen for most real-world"},{"line_number":14,"context_line":"installations, so let\u0027s try to get back to using distro packages. We\u0027ll"},{"line_number":15,"context_line":"still use the source install for RHEL/Ubuntu, it remains to be tested"},{"line_number":16,"context_line":"whether we can get back to using distro versions there, too."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Depends-On: https://review.opendev.org/731159"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":24,"id":"ff570b3c_670daee7","line":15,"range":{"start_line":15,"start_character":37,"end_line":15,"end_character":44},"updated":"2020-06-10 07:20:37.000000000","message":"? I thought the change is about not having the source install on Ubuntu (and I don\u0027t think \"RHEL over Ubuntu\" would be a proper interpretation either?).","commit_id":"21b34de820cf303c000251cf417dbaaf76fb5ef2"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"6d9c54d852cf7495c6f696971a0defbb43636b36","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Also it doesn\u0027t match what will happen for most real-world"},{"line_number":14,"context_line":"installations, so let\u0027s try to get back to using distro packages. We\u0027ll"},{"line_number":15,"context_line":"still use the source install for RHEL/Ubuntu, it remains to be tested"},{"line_number":16,"context_line":"whether we can get back to using distro versions there, too."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Depends-On: https://review.opendev.org/731159"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":24,"id":"ff570b3c_7c7a51f0","line":15,"range":{"start_line":15,"start_character":37,"end_line":15,"end_character":44},"in_reply_to":"ff570b3c_670daee7","updated":"2020-06-11 05:11:38.000000000","message":"Yeah, I failed to update this properly when I merged the Fedora and Ubuntu patches, which is needed now since things fail with just one of them.","commit_id":"21b34de820cf303c000251cf417dbaaf76fb5ef2"}],"lib/apache":[{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"12f4fa22c53576d65ef152252bd08c97ac742b98","unresolved":false,"context_lines":[{"line_number":96,"context_line":"                        uwsgi-plugin-python \\"},{"line_number":97,"context_line":"                        uwsgi-plugin-python3 \\"},{"line_number":98,"context_line":"                        libapache2-mod-proxy-uwsgi"},{"line_number":99,"context_line":"        if python3_enabled; then"},{"line_number":100,"context_line":"            UWSGI_PYTHON_PLUGIN\u003dpython3"},{"line_number":101,"context_line":"        else"},{"line_number":102,"context_line":"            UWSGI_PYTHON_PLUGIN\u003dpython"}],"source_content_type":"application/x-shellscript","patch_set":19,"id":"ff570b3c_0e94c6e7","line":99,"updated":"2020-05-26 15:25:43.000000000","message":"NIT can we put this outside the distro check conditional?","commit_id":"c35e46f5f9093bafea14c617721bbea8fdf9133e"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d09a94a496b8bab0d0e21fc79ee0b4c9a1956212","unresolved":false,"context_lines":[{"line_number":96,"context_line":"                        uwsgi-plugin-python \\"},{"line_number":97,"context_line":"                        uwsgi-plugin-python3 \\"},{"line_number":98,"context_line":"                        libapache2-mod-proxy-uwsgi"},{"line_number":99,"context_line":"        if python3_enabled; then"},{"line_number":100,"context_line":"            UWSGI_PYTHON_PLUGIN\u003dpython3"},{"line_number":101,"context_line":"        else"},{"line_number":102,"context_line":"            UWSGI_PYTHON_PLUGIN\u003dpython"}],"source_content_type":"application/x-shellscript","patch_set":19,"id":"ff570b3c_b55365e2","line":99,"in_reply_to":"ff570b3c_0e94c6e7","updated":"2020-05-26 18:40:08.000000000","message":"Done","commit_id":"c35e46f5f9093bafea14c617721bbea8fdf9133e"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"8eb682fa29f481f7af143df8c83a062b3f558991","unresolved":false,"context_lines":[{"line_number":110,"context_line":"        install_package uwsgi \\"},{"line_number":111,"context_line":"                        uwsgi-plugin-python3"},{"line_number":112,"context_line":"    else"},{"line_number":113,"context_line":"        # Centos actually has the module in epel, but there was a big"},{"line_number":114,"context_line":"        # push to disable epel by default. As such, compile from source"},{"line_number":115,"context_line":"        # there."},{"line_number":116,"context_line":"        local dir"},{"line_number":117,"context_line":"        dir\u003d$(mktemp -d)"},{"line_number":118,"context_line":"        pushd $dir"}],"source_content_type":"application/x-shellscript","patch_set":24,"id":"ff570b3c_2703b6c0","line":115,"range":{"start_line":113,"start_character":10,"end_line":115,"end_character":16},"updated":"2020-06-10 07:20:37.000000000","message":"I wonder if RDO would not help us?","commit_id":"21b34de820cf303c000251cf417dbaaf76fb5ef2"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"6d9c54d852cf7495c6f696971a0defbb43636b36","unresolved":false,"context_lines":[{"line_number":110,"context_line":"        install_package uwsgi \\"},{"line_number":111,"context_line":"                        uwsgi-plugin-python3"},{"line_number":112,"context_line":"    else"},{"line_number":113,"context_line":"        # Centos actually has the module in epel, but there was a big"},{"line_number":114,"context_line":"        # push to disable epel by default. As such, compile from source"},{"line_number":115,"context_line":"        # there."},{"line_number":116,"context_line":"        local dir"},{"line_number":117,"context_line":"        dir\u003d$(mktemp -d)"},{"line_number":118,"context_line":"        pushd $dir"}],"source_content_type":"application/x-shellscript","patch_set":24,"id":"ff570b3c_3c4299ac","line":115,"range":{"start_line":113,"start_character":10,"end_line":115,"end_character":16},"in_reply_to":"ff570b3c_2703b6c0","updated":"2020-06-11 05:11:38.000000000","message":"This block isn\u0027t touched by this patch, just moved a bit. Fixing up installation for Centos would be a good topic for another followup, but since I know next to nothing about it, someone else would have to do it.","commit_id":"21b34de820cf303c000251cf417dbaaf76fb5ef2"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"a9054bcaafa9a3c492c86ebe51800622fdcc3bf1","unresolved":false,"context_lines":[{"line_number":119,"context_line":"        pip_install uwsgi"},{"line_number":120,"context_line":"        pip download uwsgi -c $REQUIREMENTS_DIR/upper-constraints.txt"},{"line_number":121,"context_line":"        local uwsgi"},{"line_number":122,"context_line":"        uwsgi\u003d$(ls uwsgi*)"},{"line_number":123,"context_line":"        tar xvf $uwsgi"},{"line_number":124,"context_line":"        cd uwsgi*/apache2"},{"line_number":125,"context_line":"        sudo $apxs -i -c mod_proxy_uwsgi.c"}],"source_content_type":"application/x-shellscript","patch_set":27,"id":"bf51134e_c620dc4c","line":122,"updated":"2020-07-06 15:57:30.000000000","message":"I\u0027m not sure if this works because the downloaded file has capital letters that aren\u0027t picked up:\n\n    Collecting uwsgi\n    Downloading https://files.pythonhosted.org/packages/c7/75/45234f7b441c59b1eefd31ba3d1041a7e3c89602af24488e2a22e11e7259/uWSGI-2.0.19.1.tar.gz (803kB)\n\n        1% |▍                               | 10kB 47.4MB/s eta 0:00:01\n        100% |████████████████████████████████| 808kB 1.4MB/s\n    Saved ./uWSGI-2.0.19.1.tar.gz\n    Successfully downloaded uwsgi\n    You are using pip version 9.0.3, however version 20.1.1 is available.\n    You should consider upgrading via the \u0027pip install --upgrade pip\u0027 command.\n    +lib/apache:install_apache_uwsgi:121       local uwsgi\n    ++lib/apache:install_apache_uwsgi:122       ls \u0027uwsgi*\u0027\n    ls: cannot access uwsgi*: No such file or directory\n    +lib/apache:install_apache_uwsgi:122       uwsgi\u003d\n    +lib/apache:install_apache_uwsgi:1         exit_trap\n    +stack.sh:exit_trap:528                    local r\u003d2","commit_id":"2d903568ed4158aa668bbda6986307a8780c71a4"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"826cb5841145efff1062a1317a4f1fa22123ba0b","unresolved":false,"context_lines":[{"line_number":121,"context_line":"        local uwsgi"},{"line_number":122,"context_line":"        uwsgi\u003d$(ls uwsgi*)"},{"line_number":123,"context_line":"        tar xvf $uwsgi"},{"line_number":124,"context_line":"        cd uwsgi*/apache2"},{"line_number":125,"context_line":"        sudo $apxs -i -c mod_proxy_uwsgi.c"},{"line_number":126,"context_line":"        popd"},{"line_number":127,"context_line":"        # delete the temp directory"}],"source_content_type":"application/x-shellscript","patch_set":27,"id":"bf51134e_014abe53","line":124,"updated":"2020-07-06 16:24:01.000000000","message":"ditto on the cd","commit_id":"2d903568ed4158aa668bbda6986307a8780c71a4"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"4417ff9649b5866daf4546f4220b7bcb4319a95c","unresolved":false,"context_lines":[{"line_number":121,"context_line":"        local uwsgi"},{"line_number":122,"context_line":"        uwsgi\u003d$(ls uwsgi*)"},{"line_number":123,"context_line":"        tar xvf $uwsgi"},{"line_number":124,"context_line":"        cd uwsgi*/apache2"},{"line_number":125,"context_line":"        sudo $apxs -i -c mod_proxy_uwsgi.c"},{"line_number":126,"context_line":"        popd"},{"line_number":127,"context_line":"        # delete the temp directory"}],"source_content_type":"application/x-shellscript","patch_set":27,"id":"bf51134e_2c7b79f4","line":124,"in_reply_to":"bf51134e_014abe53","updated":"2020-07-06 17:31:45.000000000","message":"I take that back, it\u0027s not a capitalization error on the \"cd\" -- it failed because the .tar.gz is extracted in the local directory so \"apache2\" is in the current working directory, not under uWSGI*","commit_id":"2d903568ed4158aa668bbda6986307a8780c71a4"}]}