From: Hans-Christoph Steiner <hans@eds.org>
Date: Thu, 11 Feb 2016 23:27:17 +0000 (+0100)
Subject: fix errors in pre-commit hook when run in test suite
X-Git-Tag: 0.6.0~11^2~1
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=dfca237329439b8bb62e2142a60113d563f89e4a;p=fdroidserver.git

fix errors in pre-commit hook when run in test suite
---

diff --git a/hooks/pre-commit b/hooks/pre-commit
index d33c7018..6dd0049d 100755
--- a/hooks/pre-commit
+++ b/hooks/pre-commit
@@ -83,13 +83,13 @@ else
 	err "pep8 is not installed!"
 fi
 
-if [ ! -z $PY_FILES $PY_TEST_FILES ]; then
+if [ "$PY_FILES $PY_TEST_FILES" != " " ]; then
     if ! $PYFLAKES $PY_FILES $PY_TEST_FILES; then
 	err "pyflakes tests failed!"
     fi
 fi
 
-if [ ! -z $PY_FILES ]; then
+if [ "$PY_FILES" != "" ]; then
     if ! $PEP8 --ignore=$PEP8_IGNORE $PY_FILES; then
 	err "pep8 tests failed!"
     fi
@@ -98,7 +98,7 @@ fi
 # The tests use a little hack in order to cleanly import the fdroidserver
 # package locally like a regular package.  pep8 doesn't see that, so this
 # makes pep8 skip E402 on the test files that need that hack.
-if [ ! -z $PY_TEST_FILES ]; then
+if [ "$PY_TEST_FILES" != "" ]; then
     if ! $PEP8 --ignore=$PEP8_IGNORE,E402 $PY_TEST_FILES; then
 	err "pep8 tests failed!"
     fi