[libvirt] [PATCH python 13/14] sanitytest: Fix broken comparison between int and string

2013-12-09 Thread Daniel P. Berrange
From: Daniel P. Berrange berra...@redhat.com

Python2 was forgiving of a comparison between an int and string
but Python3 gets very upset.

Signed-off-by: Daniel P. Berrange berra...@redhat.com
---
 sanitytest.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sanitytest.py b/sanitytest.py
index eb4caee..50e4069 100644
--- a/sanitytest.py
+++ b/sanitytest.py
@@ -103,7 +103,7 @@ for cname in wantfunctions:
 found = True
 if name not in basicklassmap:
 basicklassmap[name] = [klassname, name[klen:], cname]
-elif len(basicklassmap[name])  klassname:
+elif len(basicklassmap[name])  klen:
 basicklassmap[name] = [klassname, name[klen:], cname]
 
 # Anything which can't map to a class goes into the
-- 
1.8.3.1

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


Re: [libvirt] [PATCH python 13/14] sanitytest: Fix broken comparison between int and string

2013-12-09 Thread Doug Goldstein
On Mon, Dec 9, 2013 at 9:15 AM, Daniel P. Berrange berra...@redhat.com wrote:
 From: Daniel P. Berrange berra...@redhat.com

 Python2 was forgiving of a comparison between an int and string
 but Python3 gets very upset.

 Signed-off-by: Daniel P. Berrange berra...@redhat.com
 ---
  sanitytest.py | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/sanitytest.py b/sanitytest.py
 index eb4caee..50e4069 100644
 --- a/sanitytest.py
 +++ b/sanitytest.py
 @@ -103,7 +103,7 @@ for cname in wantfunctions:
  found = True
  if name not in basicklassmap:
  basicklassmap[name] = [klassname, name[klen:], cname]
 -elif len(basicklassmap[name])  klassname:
 +elif len(basicklassmap[name])  klen:
  basicklassmap[name] = [klassname, name[klen:], cname]

  # Anything which can't map to a class goes into the
 --
 1.8.3.1

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

ACK.

-- 
Doug Goldstein

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