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 b8e3dca560e4445423978253a5fc07a71cea9839..20b20aae0b1f0b736b45cedb9d3a4a7df4da7fcd 100644 (file)
--- 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 e5c8626777351ca87b30283982c905aeb26f8660..8ac7c24fcc09183919d53f556ea3bd3b5e454478 100644 (file)
--- 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.