You can join patches 7 to 10 into one single patch. It is easier to review as they modifies the same file for the same proposal. On 05/30/2016 04:10 AM, archus@linux.vnet.ibm.com wrote:

From: Archana Singh <archus@linux.vnet.ibm.com>

Added check-fvt to run FVT testcases using make and venv dir to be cleaned.

Signed-off-by: Archana Singh <archus@linux.vnet.ibm.com> --- Makefile.am | 6 ++++++ 1 file changed, 6 insertions(+)

diff --git a/Makefile.am b/Makefile.am index 325d0c9..9827e66 100644 --- a/Makefile.am +++ b/Makefile.am @@ -52,6 +52,7 @@ check-local: find . -path './.git' -prune -o \ -name '*.py' -o -name '*.py.in' | \ xargs $(PYFLAKES) | \ + grep -w -v "venv" | \ while read LINE; do echo "$$LINE"; false; done \ else \ find . -name '*.py' -o -name '*.py.in' | \ @@ -70,6 +71,10 @@ check-local: /bin/bash check-IBM-license-header.sh ; \ fi

+check-fvt: + tests/fvt/run_tests.sh + rm -fr venv + # Link built mo files in the source tree to enable use of translations from # within the source tree all-local: @@ -185,5 +190,6 @@ VERSION:

clean-local: rm -rf mo rpm + rm -rf mo rpm tests/fvt/venv

CLEANFILES = wok.spec `find "$(top_srcdir)" -type f -name "*.pyc" -print`