[libvirt] [PATCH python] Fix virStream.sendAll for Python 3
Xavier Fernandez
xav.fernandez at gmail.com
Wed May 17 12:27:56 UTC 2017
Great !
Thanks for the merge and your work :)
Cheers,
Xavier
On Wed, May 17, 2017 at 9:12 AM, Martin Kletzander <mkletzan at redhat.com>
wrote:
> On Fri, May 05, 2017 at 04:39:34PM +0200, Xavier Fernandez wrote:
>
>> In Python 3, if the file is open in binary mode, ``got`` will end up
>> being equal to b"" and b"" != "" in Python 3.
>> ---
>> libvirt-override-virStream.py | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>>
> Makes sense, I also stumbled upon something similar. ACK and I will
> push this in a while, thanks for the patch. I'll reword the commit
> message a tiny bit.
>
> diff --git a/libvirt-override-virStream.py b/libvirt-override-virStream.py
>> index c328154..2e77cc7 100644
>> --- a/libvirt-override-virStream.py
>> +++ b/libvirt-override-virStream.py
>> @@ -83,7 +83,7 @@
>> pass
>> raise e
>>
>> - if got == "":
>> + if not got:
>> break
>>
>> ret = self.send(got)
>> --
>> 2.12.2
>>
>> --
>> libvir-list mailing list
>> libvir-list at redhat.com
>> https://www.redhat.com/mailman/listinfo/libvir-list
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170517/c0ad9803/attachment-0001.htm>
More information about the libvir-list
mailing list