On Fri, Sep 20, 2013 at 05:31:59AM -0400, Timothy Asir Jeyasingh wrote:
Hi All,
The vdsm (master branch) rpm build failed with the following error:
find . -path './.git' -prune -type f -o \
-name '*.py' -o -name '*.py.in' | xargs /usr/bin/pyflakes | \
grep -w -v "\./vdsm/storage/lvm\.py.*: list comprehension redefines 'lv'
from line .*" | \
while read LINE; do echo "$LINE"; false; done
/usr/bin/pep8 --version
1.4.6
/usr/bin/pep8 --exclude="config.py,constants.py" --filename
'*.py,*.py.in' \
client lib/cpopen/*.py lib/vdsm/*.py lib/vdsm/*.py.in tests vds_bootstrap vdsm-tool
vdsm/*.py vdsm/*.py.in vdsm/netconf vdsm/sos/vdsm.py.in vdsm/storage vdsm/vdsm vdsm_api
vdsm_hooks vdsm_reg
lib/vdsm/SecureXMLRPCServer.py:143:1: E303 too many blank lines (3)
make[3]: *** [check-local] Error 1
make[3]: Leaving directory `/home/timothy/rpmbuild/BUILD/vdsm-4.12.0'
make[2]: *** [check-am] Error 2
make[2]: Leaving directory `/home/timothy/rpmbuild/BUILD/vdsm-4.12.0'
make[1]: *** [check-recursive] Error 1
make[1]: Leaving directory `/home/timothy/rpmbuild/BUILD/vdsm-4.12.0'
error: Bad exit status from /var/tmp/rpm-tmp.PcZNhb (%check)
Sorry about that. The pep8 failure was masked by another failure of
apiTests.JsonRawTest.testBadMethod
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4499/testReport/junit...
Saggi, could you look into these recurring breakage?
As David Caro explains in a comment to
http://gerrit.ovirt.org/#/c/19332/ , I carelessly ignored the pep8
failure in Juan's patch.
Problem solved by Juan's
http://gerrit.ovirt.org/19408 .
Thanks, and sorry for the noise.