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/apiTests/JsonRawTest/testBadMethod/
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.
___
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel