From: Tarek Ziadé <ziade.tarek@gmail.com>
Date: Sun, 29 Nov 2009 22:20:30 +0000 (+0000)
Subject: Fixed #7408: dropped group ownership checking because it relies on os-specific rules
X-Git-Tag: v2.7a1~23
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=a5076a2543e9d52128b875bc4b3b0521ed3a2b12;p=thirdparty%2FPython%2Fcpython.git

Fixed #7408: dropped group ownership checking because it relies on os-specific rules
---

diff --git a/Lib/distutils/tests/test_sdist.py b/Lib/distutils/tests/test_sdist.py
index b8e3dca560e4..20b20aae0b1f 100644
--- a/Lib/distutils/tests/test_sdist.py
+++ b/Lib/distutils/tests/test_sdist.py
@@ -336,10 +336,13 @@ class SDistTestCase(PyPIRCCommandTestCase):
         # making sure we have the good rights
         archive_name = join(self.tmp_dir, 'dist', 'fake-1.0.tar.gz')
         archive = tarfile.open(archive_name)
+
+        # note that we are not testing the group ownership here
+        # because, depending on the platforms and the container
+        # rights (see #7408)
         try:
             for member in archive.getmembers():
                 self.assertEquals(member.uid, os.getuid())
-                self.assertEquals(member.gid, os.getgid())
         finally:
             archive.close()
 
diff --git a/Misc/NEWS b/Misc/NEWS
index e5c862677735..8ac7c24fcc09 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -487,6 +487,10 @@ Core and Builtins
 Library
 -------
 
+- Issue #7408: Fixed distutils.tests.sdist so it doesn't check for group
+  ownership when the group is not forced, because the group may be different
+  from the user's group and inherit from its container when the test is run.
+
 - Issue #1515: Enable use of deepcopy() with instance methods.  Patch by
   Robert Collins.