21230: Remove pip install from get-package-version
authorBrett Smith <brett.smith@curii.com>
Fri, 12 Jan 2024 14:39:41 +0000 (09:39 -0500)
committerBrett Smith <brett.smith@curii.com>
Fri, 12 Jan 2024 14:39:41 +0000 (09:39 -0500)
Reasons to remove this: it can't work on Debian 12+, and the code has
obviously gotten stale with its original rationale ('setuptools<45' is
definitely not "latest setuptools").

Nobody can tell me what context this runs in. I'm going to hope that it
runs in some context where setuptools is already available. If it turns
out I'm wrong, then hopefully that will break the build in a way that
lets me assess our options.

Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>

build/get-package-version.sh

index 5147f7bba6adc9bbd0795ad76f6b6e16a8fcb130..390b5dd828f456fa1e8aa0867968814295f888fd 100755 (executable)
@@ -44,12 +44,6 @@ elif [[ "$TYPE_LANG" == "python3" ]]; then
 
   rm -rf dist/*
 
-  # Get the latest setuptools
-  if ! pip3 install $DASHQ_UNLESS_DEBUG $CACHE_FLAG -U 'setuptools<45'; then
-    echo "Error, unable to upgrade setuptools with"
-    echo "  pip3 install $DASHQ_UNLESS_DEBUG $CACHE_FLAG -U 'setuptools<45'"
-    exit 1
-  fi
   # filter a useless warning (when building the cwltest package) from the stderr output
   if ! python3 setup.py $DASHQ_UNLESS_DEBUG sdist 2> >(grep -v 'warning: no previously-included files matching' |grep -v 'for version number calculation'); then
     echo "Error, unable to run python3 setup.py sdist for $SRC_PATH"