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 [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
