changeset:   99284:e7d1df56f70b
branch:      3.4
parent:      99280:d51d420f3e9d
user:        Serhiy Storchaka <storchaka@gmail.com>
date:        Sun Nov 22 14:49:58 2015 +0200
files:       Lib/test/test_shutil.py Lib/zipfile.py Misc/ACKS Misc/NEWS
description:
Issue #25624: ZipFile now always writes a ZIP_STORED header for directory
entries.  Patch by Dingyuan Wang.


diff -r d51d420f3e9d -r e7d1df56f70b Lib/test/test_shutil.py
--- a/Lib/test/test_shutil.py	Sun Nov 22 12:18:38 2015 +0200
+++ b/Lib/test/test_shutil.py	Sun Nov 22 14:49:58 2015 +0200
@@ -1099,6 +1099,29 @@
             names2 = zf.namelist()
         self.assertEqual(sorted(names), sorted(names2))
 
+    @requires_zlib
+    @unittest.skipUnless(ZIP_SUPPORT, 'Need zip support to run')
+    @unittest.skipUnless(shutil.which('unzip'),
+                         'Need the unzip command to run')
+    def test_unzip_zipfile(self):
+        root_dir, base_dir = self._create_files()
+        base_name = os.path.join(self.mkdtemp(), 'archive')
+        archive = make_archive(base_name, 'zip', root_dir, base_dir)
+
+        # check if ZIP file  was created
+        self.assertEqual(archive, base_name + '.zip')
+        self.assertTrue(os.path.isfile(archive))
+
+        # now check the ZIP file using `unzip -t`
+        zip_cmd = ['unzip', '-t', archive]
+        with support.change_cwd(root_dir):
+            try:
+                subprocess.check_output(zip_cmd, stderr=subprocess.STDOUT)
+            except subprocess.CalledProcessError as exc:
+                details = exc.output.decode(errors="replace")
+                msg = "{}\n\n**Unzip Output**\n{}"
+                self.fail(msg.format(exc, details))
+
     def test_make_archive(self):
         tmpdir = self.mkdtemp()
         base_name = os.path.join(tmpdir, 'archive')
diff -r d51d420f3e9d -r e7d1df56f70b Lib/zipfile.py
--- a/Lib/zipfile.py	Sun Nov 22 12:18:38 2015 +0200
+++ b/Lib/zipfile.py	Sun Nov 22 14:49:58 2015 +0200
@@ -1337,7 +1337,9 @@
             arcname += '/'
         zinfo = ZipInfo(arcname, date_time)
         zinfo.external_attr = (st[0] & 0xFFFF) << 16      # Unix attributes
-        if compress_type is None:
+        if isdir:
+            zinfo.compress_type = ZIP_STORED
+        elif compress_type is None:
             zinfo.compress_type = self.compression
         else:
             zinfo.compress_type = compress_type
diff -r d51d420f3e9d -r e7d1df56f70b Misc/ACKS
--- a/Misc/ACKS	Sun Nov 22 12:18:38 2015 +0200
+++ b/Misc/ACKS	Sun Nov 22 14:49:58 2015 +0200
@@ -1471,6 +1471,7 @@
 Larry Wall
 Kevin Walzer
 Rodrigo Steinmuller Wanderley
+Dingyuan Wang
 Ke Wang
 Greg Ward
 Tom Wardill
diff -r d51d420f3e9d -r e7d1df56f70b Misc/NEWS
--- a/Misc/NEWS	Sun Nov 22 12:18:38 2015 +0200
+++ b/Misc/NEWS	Sun Nov 22 14:49:58 2015 +0200
@@ -106,6 +106,9 @@
 Library
 -------
 
+- Issue #25624: ZipFile now always writes a ZIP_STORED header for directory
+  entries.  Patch by Dingyuan Wang.
+
 - Issue #25583: Avoid incorrect errors raised by os.makedirs(exist_ok=True)
   when the OS gives priority to errors such as EACCES over EEXIST.