Merge ~pappacena/turnip:make-target-py3-check into turnip:master

Proposed by Thiago F. Pappacena
Status: Merged
Approved by: Thiago F. Pappacena
Approved revision: ed898b054b4cf8cb081ec9a17582cc15b872f590
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~pappacena/turnip:make-target-py3-check
Merge into: turnip:master
Diff against target: 52 lines (+9/-1)
2 files modified
.gitignore (+1/-0)
Makefile (+8/-1)
Reviewer Review Type Date Requested Status
Colin Watson (community) Approve
Review via email: mp+383759@code.launchpad.net

Commit message

Adding Makefile rule to run `make check-python-compat` and run tests on both python 2 and python 3 envs.

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) :
review: Needs Information
1a866e1... by Thiago F. Pappacena

Keeping python2 and python3 envs simultaneously

Revision history for this message
Thiago F. Pappacena (pappacena) wrote :

cjwatson, pushed the changes to keep both venvs at the same time. Let me know if you would like to review that.

75589bf... by Thiago F. Pappacena

Merge branch 'master' into make-target-py3-check

Revision history for this message
Colin Watson (cjwatson) :
review: Approve
Revision history for this message
Colin Watson (cjwatson) :
ed898b0... by Thiago F. Pappacena

Preventing environment variables from overriding some Makefile vars

Revision history for this message
Thiago F. Pappacena (pappacena) :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/.gitignore b/.gitignore
index fb9d39c..5698ab9 100644
--- a/.gitignore
+++ b/.gitignore
@@ -6,6 +6,7 @@ dist
6download-cache6download-cache
7eggs7eggs
8env8env
9py3env
9pip-cache10pip-cache
10*.egg*11*.egg*
11*.egg-info12*.egg-info
diff --git a/Makefile b/Makefile
index c514366..9e6bb12 100644
--- a/Makefile
+++ b/Makefile
@@ -2,6 +2,7 @@
2# GNU Affero General Public License version 3 (see the file LICENSE).2# GNU Affero General Public License version 3 (see the file LICENSE).
33
4ENV := $(CURDIR)/env4ENV := $(CURDIR)/env
5PY3_ENV := $(CURDIR)/py3env
5PIP_CACHE = $(CURDIR)/pip-cache6PIP_CACHE = $(CURDIR)/pip-cache
67
7PYTHON := $(ENV)/bin/python8PYTHON := $(ENV)/bin/python
@@ -11,6 +12,7 @@ PIP := $(ENV)/bin/pip
11VIRTUALENV := virtualenv12VIRTUALENV := virtualenv
1213
13DEPENDENCIES_URL := https://git.launchpad.net/~canonical-launchpad-branches/turnip/+git/dependencies14DEPENDENCIES_URL := https://git.launchpad.net/~canonical-launchpad-branches/turnip/+git/dependencies
15PIP_SOURCE_DIR := dependencies
1416
15PIP_ARGS ?= --quiet17PIP_ARGS ?= --quiet
16ifneq ($(PIP_SOURCE_DIR),)18ifneq ($(PIP_SOURCE_DIR),)
@@ -59,7 +61,7 @@ test: $(ENV)
5961
60clean:62clean:
61 find turnip -name '*.py[co]' -exec rm '{}' \;63 find turnip -name '*.py[co]' -exec rm '{}' \;
62 rm -rf $(ENV) $(PIP_CACHE)64 rm -rf $(ENV) $(PY3_ENV) $(PIP_CACHE)
63 rm -f turnip/version_info.py65 rm -f turnip/version_info.py
6466
65dist:67dist:
@@ -80,6 +82,11 @@ pip-check: $(ENV)
8082
81check: pip-check test lint83check: pip-check test lint
8284
85check-python3:
86 $(MAKE) check VENV_ARGS="-p python3" ENV="$(PY3_ENV)"
87
88check-python-compat: check check-python3
89
83run-api: $(ENV)90run-api: $(ENV)
84 $(PSERVE) api.ini --reload91 $(PSERVE) api.ini --reload
8592

Subscribers

People subscribed via source and target branches