Re: [PATCH python] Fix virStream.sendAll for Python 3

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Great !
Thanks for the merge and your work :)

Cheers,
Xavier

On Wed, May 17, 2017 at 9:12 AM, Martin Kletzander <mkletzan@xxxxxxxxxx> 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@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux