From 5fcb5d59efedd947cbfb7bb1c628ee51037fbab2 Mon Sep 17 00:00:00 2001
From: David Pursehouse <david.pursehouse@sonymobile.com>
Date: Tue, 6 Aug 2013 17:54:50 +0900
Subject: Separate test dependency installation into its own build target

Packages required for testing are now installed via a separate build
target which is a dependency of the main packages installation target.

Change-Id: I62393e1c7f8a8ff3bf95f3ac3090e43817fbed79
---
 Makefile | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/Makefile b/Makefile
index 456f48d..f69e63e 100644
--- a/Makefile
+++ b/Makefile
@@ -24,36 +24,41 @@ all: test
 
 test: clean unittests pyflakes pep8 pep257 pylint
 
-pylint: envsetup
+pylint: testenvsetup
 	bash -c "\
           source ./pygerritenv/bin/activate && \
           git ls-files | grep \"\.py$$\" | grep -v "unittests" | \
           xargs pylint --rcfile=.pylintrc"
 
-pep257: envsetup
+pep257: testenvsetup
 	bash -c "\
           source ./pygerritenv/bin/activate && \
           git ls-files | grep \"\.py$$\" | xargs pep257"
 
-pep8: envsetup
+pep8: testenvsetup
 	bash -c "\
           source ./pygerritenv/bin/activate && \
           git ls-files | grep \"\.py$$\" | xargs pep8 --max-line-length 80"
 
-pyflakes: envsetup
+pyflakes: testenvsetup
 	bash -c "\
           source ./pygerritenv/bin/activate && \
           git ls-files | grep \"\.py$$\" | xargs pyflakes"
 
-unittests: envsetup
+unittests: testenvsetup
 	bash -c "\
           source ./pygerritenv/bin/activate && \
           python unittests.py"
 
+testenvsetup: envsetup
+	bash -c "\
+          source ./pygerritenv/bin/activate && \
+          pip install --upgrade -r test_requirements.txt"
+
 envsetup: envinit
 	bash -c "\
           source ./pygerritenv/bin/activate && \
-          pip install --upgrade -r requirements.txt -r test_requirements.txt"
+          pip install --upgrade -r requirements.txt"
 
 envinit:
 	bash -c "[ -e ./pygerritenv/bin/activate ] || virtualenv ./pygerritenv"
-- 
cgit v1.2.1