Remove tox from python-tools-install. 00/65400/2
authorAric Gardner <agardner@linuxfoundation.org>
Wed, 9 Sep 2020 16:02:15 +0000 (12:02 -0400)
committerAric Gardner <agardner@linuxfoundation.org>
Wed, 9 Sep 2020 16:08:32 +0000 (12:08 -0400)
If users need a system pip install of tox
they can include tox-install.
However, unless they need pyenv
I would suggest they use lf-env.

Order of system pip install of tox matters,
whichever one is installed most recently
is the one that is going to be called.
removing tox from python tools install
allows the install of tox in tox-install
to not be skipped, and in this way
we can ensure that the py3 version
is the once most recently installed

Issue-Id: RELENG-3189
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>
Change-Id: Ie904f42821f6598a389854e8876448237cfac3dd

shell/python-tools-install.sh

index bf1cf8e..bf9b12e 100644 (file)
@@ -29,7 +29,6 @@ python-openstackclient
 python-magnumclient
 kubernetes
 niet~=1.4.2
-tox>=3.7.0 # Tox 3.7 or greater is necessary for parallel mode support
 yq
 
 # PINNED INDIRECT DEPENDENCIES
@@ -50,25 +49,23 @@ EOF
          echo "zipp==1.1.0" >> "$requirements_file"
        fi
     fi
-    # Use `python -m pip` to upgrade to the latest pip into user site-packages
+
+    #Project may still want to use system py27
+    #pip and setuptools must be updated for py27 to work.
+    python -m pip install --user --quiet --upgrade pip
+    python -m pip install --user --quiet --no-warn-script-location --upgrade setuptools
+    python --version
+    python -m pip --version
+    python -m pip freeze
+
     python3 -m pip install --user --quiet --upgrade pip
     python3 -m pip install --user --quiet --no-warn-script-location --upgrade setuptools
     python3 -m pip install --user --quiet --no-warn-script-location --upgrade --upgrade-strategy eager -r "$requirements_file"
     # installs are silent, show version details in log
     python3 --version
-
     python3 -m pip --version
     python3 -m pip freeze
 
-    #workaround
-    #system tox is py2.7 needs updated pip and setuptools.
-    #this needs to be fixed in packer. tox is py3 on my machine
-    python -m pip install --user --quiet --upgrade pip
-    python -m pip install --user --quiet --no-warn-script-location --upgrade setuptools
-    python --version
-    python -m pip --version
-    python -m pip freeze
-
     rm -rf "$requirements_file"
     touch /tmp/pre-build-complete
 fi