From: Tarek Ziadé <ziade.tarek@gmail.com>
Date: Sun, 29 Nov 2009 22:24:57 +0000 (+0000)
Subject: Merged revisions 76588 via svnmerge from
X-Git-Tag: v3.2a1~2103
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=acafc2d3d114fbbdf65bd1d77a6bbf39b94971f6;p=python

Merged revisions 76588 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r76588 | tarek.ziade | 2009-11-29 23:20:30 +0100 (Sun, 29 Nov 2009) | 1 line

  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 c10e973ed7..e0f1e93768 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 4efc479074..96fbbce89d 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -140,6 +140,10 @@ C-API
 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 #4486: When an exception has an explicit cause, do not print its
   implicit context too.  This affects the `traceback` module as well as
   built-in exception printing.