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?
I'm using python-pep8-1.5.4-1.fc20.noarch
On 13/05/14 17:22 -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?
I found this one too and am planning to submit the same workaround. Actually you can just do (-1) without int().
pep8-1.5.6 is available in F20 repos and resolves the issue.
On 05/20/2014 09:22 PM, Adam Litke wrote:
On 13/05/14 17:22 -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?
I found this one too and am planning to submit the same workaround. Actually you can just do (-1) without int().
vdsm-devel@lists.fedorahosted.org