Skip to content

Commit f241109

Browse files
committed
Fix pip install for py3 in travis on st2client check
1 parent edbc80b commit f241109

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

Makefile

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -229,9 +229,6 @@ check-python-packages:
229229
@echo "================== CHECK PYTHON PACKAGES ===================="
230230
@echo ""
231231
test -f $(VIRTUALENV_COMPONENTS_DIR)/bin/activate || $(PYTHON_VERSION) -m venv $(VIRTUALENV_COMPONENTS_DIR) || virtualenv --python=$(PYTHON_VERSION) $(VIRTUALENV_COMPONENTS_DIR) --no-download
232-
@if [[ $(PYTHON_VERSION) == *"python3.6"* ]]; then \
233-
$(PYTHON_VERSION) -m pip install --upgrade pip; \
234-
fi
235232
@for component in $(COMPONENTS_WITHOUT_ST2TESTS); do \
236233
echo "==========================================================="; \
237234
echo "Checking component:" $$component; \
@@ -380,9 +377,6 @@ flake8: requirements .flake8
380377
@echo "==================== st2client install check ===================="
381378
@echo
382379
test -f $(VIRTUALENV_ST2CLIENT_DIR)/bin/activate || $(PYTHON_VERSION) -m venv $(VIRTUALENV_ST2CLIENT_DIR) || virtualenv --python=$(PYTHON_VERSION) $(VIRTUALENV_ST2CLIENT_DIR) --no-download
383-
@if [[ $(PYTHON_VERSION) == *"python3.6"* ]]; then \
384-
$(PYTHON_VERSION) -m pip install --upgrade pip; \
385-
fi
386380

387381
# Setup PYTHONPATH in bash activate script...
388382
# Delete existing entries (if any)
@@ -396,11 +390,19 @@ flake8: requirements .flake8
396390
touch $(VIRTUALENV_ST2CLIENT_DIR)/bin/activate
397391
chmod +x $(VIRTUALENV_ST2CLIENT_DIR)/bin/activate
398392

399-
$(VIRTUALENV_ST2CLIENT_DIR)/bin/pip install --upgrade "pip==$(PIP_VERSION)"
393+
if [[ $(PYTHON_VERSION) == *"python2.7"* ]]; then \
394+
echo 'Upgrading pip==$(PIP_VERSION) in python2.7 virtualenv' \
395+
$(VIRTUALENV_ST2CLIENT_DIR)/bin/pip install --upgrade "pip==$(PIP_VERSION)"; \
396+
else \
397+
echo 'Upgrading pip==$(PIP_VERSION) in python3.x virtualenv' \
398+
$(VIRTUALENV_ST2CLIENT_DIR)/bin/python -m pip install --user --upgrade "pip==$(PIP_VERSION)"; \
399+
fi
400+
400401
# NOTE We need to upgrade setuptools to avoid bug with dependency resolving in old versions
401402
# Setuptools 42 added support for python_requires, which is used by the configparser package,
402403
# which is required by the importlib-metadata package
403404
$(VIRTUALENV_ST2CLIENT_DIR)/bin/pip install --upgrade "setuptools==44.1.0"
405+
404406
$(VIRTUALENV_ST2CLIENT_DIR)/bin/activate; cd st2client ; ../$(VIRTUALENV_ST2CLIENT_DIR)/bin/python setup.py install ; cd ..
405407
$(VIRTUALENV_ST2CLIENT_DIR)/bin/st2 --version
406408
$(VIRTUALENV_ST2CLIENT_DIR)/bin/python -c "import st2client"

0 commit comments

Comments
 (0)