Thanks a lot for the quick fix. Regards, Tim
----- Original Message ----- > From: "Dan Kenigsberg" <[email protected]> > To: "Timothy Asir Jeyasingh" <[email protected]>, [email protected] > Cc: [email protected] > Sent: Friday, September 20, 2013 6:50:27 PM > Subject: Re: [vdsm] VDSM RPM Build failed with error > SecureXMLRPCServer.py:143:1: E303 too many blank lines > > 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
