Change in vdsm[ovirt-3.2]: Makefile.am: ignore false positive pyflakes lvm.py

dougsland at redhat.com dougsland at redhat.com
Tue Apr 23 18:26:04 UTC 2013


Douglas Schilling Landgraf has uploaded a new change for review.

Change subject: Makefile.am: ignore false positive pyflakes lvm.py
......................................................................

Makefile.am: ignore false positive pyflakes lvm.py

Temporary workaround.

Change-Id: Id8cdeb0c28c1ee5dc3a18a7688ae60fe641f9a6c
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
---
M Makefile.am
1 file changed, 6 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/84/14184/1

diff --git a/Makefile.am b/Makefile.am
index 06ffbf9..7b0d161 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -99,9 +99,14 @@
 
 PEP8_BLACKLIST = apiData.py,config.py,constants.py
 
+SKIP_PYFLAKES_ERR = "\./vdsm/storage/lvm\.py.*: list comprehension redefines \
+	'lv' from line .*"
+
 check-local:
 	find . -path './.git' -prune -type f -o \
-		-name '*.py' -o -name '*.py.in'  | xargs $(PYFLAKES)
+		-name '*.py' -o -name '*.py.in'  | xargs $(PYFLAKES) | \
+		grep -w -v $(SKIP_PYFLAKES_ERR) | \
+		while read LINE; do echo "$$LINE"; false; done
 	$(PEP8) --exclude="$(PEP8_BLACKLIST)" --filename '*.py,*.py.in' \
 		--ignore=E126,E127,E128 \
 		$(PEP8_WHITELIST)


--
To view, visit http://gerrit.ovirt.org/14184
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id8cdeb0c28c1ee5dc3a18a7688ae60fe641f9a6c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.2
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>


More information about the vdsm-patches mailing list