Merge ~doismellburning/turnip:feature/pip_args into turnip:master

Proposed by Kristian Glass
Status: Merged
Approved by: Colin Watson
Approved revision: 3f1d90886d6cbff6fe4dc90582b646dca407c3d8
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~doismellburning/turnip:feature/pip_args
Merge into: turnip:master
Diff against target: 35 lines (+5/-5)
1 file modified
Makefile (+5/-5)
Reviewer Review Type Date Requested Status
Colin Watson (community) Approve
Review via email: mp+378318@code.launchpad.net

Commit message

Rename PIP_CACHE_ARGS to PIP_ARGS

PIP_CACHE_ARGS isn't just used for interacting with the pip cache,
it's currently used for all arguments including being the control
interface for pip verbosity.

Renaming it to PIP_ARGS reflects its general nature and broad usage.

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/Makefile b/Makefile
index 9661f70..689ea5e 100644
--- a/Makefile
+++ b/Makefile
@@ -12,9 +12,9 @@ VIRTUALENV := virtualenv
1212
13DEPENDENCIES_URL := https://git.launchpad.net/~canonical-launchpad-branches/turnip/+git/dependencies13DEPENDENCIES_URL := https://git.launchpad.net/~canonical-launchpad-branches/turnip/+git/dependencies
1414
15PIP_CACHE_ARGS ?= -q15PIP_ARGS ?= -q
16ifneq ($(PIP_SOURCE_DIR),)16ifneq ($(PIP_SOURCE_DIR),)
17override PIP_CACHE_ARGS += --no-index --find-links=file://$(realpath $(PIP_SOURCE_DIR))/17override PIP_ARGS += --no-index --find-links=file://$(realpath $(PIP_SOURCE_DIR))/
18endif18endif
1919
20# Create archives in labelled directories (e.g.20# Create archives in labelled directories (e.g.
@@ -50,8 +50,8 @@ endif
50 echo 'find_links = file://$(realpath $(PIP_SOURCE_DIR))/') \50 echo 'find_links = file://$(realpath $(PIP_SOURCE_DIR))/') \
51 >$(ENV)/.pydistutils.cfg51 >$(ENV)/.pydistutils.cfg
52 $(VIRTUALENV) --never-download $(ENV)52 $(VIRTUALENV) --never-download $(ENV)
53 $(PIP) install $(PIP_CACHE_ARGS) -r bootstrap-requirements.txt53 $(PIP) install $(PIP_ARGS) -r bootstrap-requirements.txt
54 $(PIP) install $(PIP_CACHE_ARGS) -c requirements.txt \54 $(PIP) install $(PIP_ARGS) -c requirements.txt \
55 -e '.[test,deploy]'55 -e '.[test,deploy]'
5656
57test: $(ENV)57test: $(ENV)
@@ -85,7 +85,7 @@ run-pack: $(ENV)
8585
86$(PIP_CACHE): $(ENV)86$(PIP_CACHE): $(ENV)
87 mkdir -p $(PIP_CACHE)87 mkdir -p $(PIP_CACHE)
88 $(PIP) install $(PIP_CACHE_ARGS) -d $(PIP_CACHE) \88 $(PIP) install $(PIP_ARGS) -d $(PIP_CACHE) \
89 -r bootstrap-requirements.txt \89 -r bootstrap-requirements.txt \
90 -r requirements.txt90 -r requirements.txt
9191

Subscribers

People subscribed via source and target branches