commit 360b3c21f5df26569be8b6431c859416d0f8f333
author Charles-François Natali <neologix@free.fr> Wed May 02 20:50:13 2012 +0200
committer Charles-François Natali <neologix@free.fr> Wed May 02 20:50:13 2012 +0200
tree b865c6e0f977f0e945c18c4fcd568b68b6eccd8c
parent d59240de8311a493ead948f382cc07e0d70fb1c1
parent 39687ee9db2303cabdf1b349ffaa60df1c01ba9d

os.popen().close() returns None on success, not 0...

Lib/test/test_posix.py

diff --git a/Lib/test/test_posix.py b/Lib/test/test_posix.py
index 5ed2a0f..21473fc 100644
--- a/Lib/test/test_posix.py
+++ b/Lib/test/test_posix.py
@@ -630,7 +630,7 @@
             groups = idg.read().strip()
             ret = idg.close()
 
-        if ret != 0 or not groups:
+        if ret != None or not groups:
             raise unittest.SkipTest("need working 'id -G'")
 
         self.assertEqual(
@@ -644,7 +644,7 @@
             groups = idg.read().strip()
             ret = idg.close()
 
-        if ret != 0 or not groups:
+        if ret != None or not groups:
             raise unittest.SkipTest("need working 'id -G'")
 
         # 'id -G' and 'os.getgroups()' should return the same