On Tue, May 13, 2014 at 5:43 PM, Dan Kenigsberg <dan...@redhat.com> wrote: > On Tue, May 13, 2014 at 05:22:15PM -0300, Amador Pahim wrote: >> Building vdsm/master in F20, I've got: >> >> ./vdsm/virt/migration.py:223:19: E225 missing whitespace around operator >> >> In vdsm/virt/migration.py: >> >> 218 e.err = (libvirt.VIR_ERR_OPERATION_ABORTED, # error code$ >> 219 libvirt.VIR_FROM_QEMU, # error >> domain$ >> 220 'operation aborted', # error >> message$ >> 221 libvirt.VIR_ERR_WARNING, # error >> level$ >> 222 '', '', '', # str1, str2, >> str3,$ >> 223 -1, -1) # int1, int2$ >> 224 raise e$ >> >> pep8 is not accepting negative integer. Instead, it is handling the >> minus sign as an operator. Quick workaround is change -1 to int(-1). >> Is this a known? > > Now it is ;-) > >> >> I'm using python-pep8-1.5.4-1.fc20.noarch > > It seems like a bug in pep8 tool, and I see that it does not reproduce > with v1.5.6 thereof. > > I'm adding Matthias who may agree to backport the fix to Fedora 20 (or > upgrade completely to 1.5.6)
Great, thank you. -- Pahim _______________________________________________ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel