Change in vdsm[master]: avoid running pep8 and pyflakes on py.in files

ybronhei at redhat.com ybronhei at redhat.com
Tue Dec 8 13:40:55 UTC 2015


Yaniv Bronhaim has uploaded a new change for review.

Change subject: avoid running pep8 and pyflakes on py.in files
......................................................................

avoid running pep8 and pyflakes on py.in files

Change-Id: I76d5e555455d1b4acc568a42b864e220c79ab6fa
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M Makefile.am
M automation/check-patch.sh
2 files changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/01/50101/1

diff --git a/Makefile.am b/Makefile.am
index bce25f4..548436a 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -105,7 +105,7 @@
 pyflakes:
 	python -c 'import pyflakes; print("pyflakes-%s" % pyflakes.__version__)'
 	( find . -path './.git' -prune -type f -o \
-		-name '*.py' -o -name '*.py.in' && \
+		-name '*.py' && \
 		echo $(WHITELIST) ) | xargs $(PYFLAKES) | \
 		grep -w -v $(SKIP_PYFLAKES_ERR) | \
 		while read LINE; do echo "$$LINE"; false; done
@@ -116,7 +116,7 @@
 	for x in $(PEP8_BLACKLIST); do \
 	    exclude="$${exclude},$${x}" ; \
 	done ; \
-	$(PEP8) --exclude="$${exclude}" --filename '*.py,*.py.in' . \
+	$(PEP8) --exclude="$${exclude}" --filename '*.py' . \
 	$(WHITELIST)
 
 .PHONY: python3
diff --git a/automation/check-patch.sh b/automation/check-patch.sh
index 259481e..ab2be2f 100755
--- a/automation/check-patch.sh
+++ b/automation/check-patch.sh
@@ -5,7 +5,7 @@
 set -xe
 
 ./autogen.sh --system --enable-hooks
-
+make all
 make check NOSE_WITH_COVERAGE=1 NOSE_COVER_PACKAGE="$PWD/vdsm,$PWD/lib"
 
 ./automation/build-artifacts.sh


-- 
To view, visit https://gerrit.ovirt.org/50101
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I76d5e555455d1b4acc568a42b864e220c79ab6fa
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>


More information about the vdsm-patches mailing list