Thanks a lot for the quick fix.

Regards,
Tim

----- Original Message -----
> From: "Dan Kenigsberg" <dan...@redhat.com>
> To: "Timothy Asir Jeyasingh" <tjeya...@redhat.com>, smizr...@redhat.com
> Cc: vdsm-devel@lists.fedorahosted.org
> 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
vdsm-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel

Reply via email to