Skip to content

Commit

Permalink
Merge pull request #1288 from finos/fix-py36-azure
Browse files Browse the repository at this point in the history
Fix #1244: make test_wheels script use python -m pip
  • Loading branch information
texodus authored Jan 14, 2021
2 parents 4074a33 + 0d070a1 commit acbaee7
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion python/perspective/scripts/test_wheels.sh
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,10 @@ source ./bin/activate
echo "${PYTHON_INTERPRETER} virtualenv activated"
cd ..

echo "Upgrading pip for ${PYTHON_INTERPRETER}..."
${PYTHON_INTERPRETER} -m pip install --upgrade pip

echo "Looking for wheels..."
cd ${HERE}/../wheelhouse

# Look for wheels based on Python version/platform
Expand All @@ -89,13 +93,15 @@ IFS=${SAVEIFS}

for wheel in ${WHEELS_LIST}; do
echo "Installing ${wheel}..."
pip install --force-reinstall ${wheel}
${PYTHON_INTERPRETER} -m pip install --force-reinstall ${wheel}

echo "-----------------------"
echo "Testing ${wheel}"

IS_LIBPSP=$(${PYTHON_INTERPRETER} -c "import perspective;print(perspective.is_libpsp())")

echo "-----------------------"

if [ "${IS_LIBPSP}" != "True" ]; then
echo "${wheel} failed to import with error: ${IS_LIBPSP}"
echo "${VENV_DIR} has been preserved for debugging"
Expand Down

0 comments on commit acbaee7

Please sign in to comment.