Issue #19921: When Path.mkdir() is called with parents=True, any missing parent is...

Patch by Serhiy.

diff --git a/Doc/library/pathlib.rst b/Doc/library/pathlib.rst
index a8c1dc9e803446a102234fd14b859ae400cd4829..7148667486e453eb8486a7767b9355c0ba7dfb7b 100644 (file)
--- a/Doc/library/pathlib.rst
+++ b/Doc/library/pathlib.rst
@@ -768,7 +768,10 @@ call fails (for example because the path doesn't exist):
    and access flags.  If the path already exists, :exc:`OSError` is raised.
 
    If *parents* is true, any missing parents of this path are created
-   as needed.  If *parents* is false (the default), a missing parent raises
+   as needed; they are created with the default permissions without taking
+   *mode* into account (mimicking the POSIX ``mkdir -p`` command).
+
+   If *parents* is false (the default), a missing parent raises
    :exc:`OSError`.
 
 

diff --git a/Lib/pathlib.py b/Lib/pathlib.py
index 9b4fde1d156b82ea81d30f5dab04dadc66fcdb2d..18facd9e0d0bdc90f5f8bb3a800836537e259ee0 100644 (file)
--- a/Lib/pathlib.py
+++ b/Lib/pathlib.py
@@ -1101,7 +1101,7 @@ class Path(PurePath):
             except OSError as e:
                 if e.errno != ENOENT:
                     raise
-                self.parent.mkdir(mode, True)
+                self.parent.mkdir(parents=True)
                 self._accessor.mkdir(self, mode)
 
     def chmod(self, mode):

diff --git a/Lib/test/test_pathlib.py b/Lib/test/test_pathlib.py
index 0ad77e052967ecdf4bcde8e1016480aa7a85bb5e..bdcba9fa6f18140a0ac50baefbd611cef701333b 100755 (executable)
--- a/Lib/test/test_pathlib.py
+++ b/Lib/test/test_pathlib.py
@@ -1478,7 +1478,6 @@ class _BasePathTest(object):
         with self.assertRaises(OSError) as cm:
             p.mkdir()
         self.assertEqual(cm.exception.errno, errno.EEXIST)
-        # XXX test `mode` arg
 
     def test_mkdir_parents(self):
         # Creating a chain of directories
@@ -1493,7 +1492,17 @@ class _BasePathTest(object):
         with self.assertRaises(OSError) as cm:
             p.mkdir(parents=True)
         self.assertEqual(cm.exception.errno, errno.EEXIST)
-        # XXX test `mode` arg
+        # test `mode` arg
+        mode = stat.S_IMODE(p.stat().st_mode) # default mode
+        p = self.cls(BASE, 'newdirD', 'newdirE')
+        p.mkdir(0o555, parents=True)
+        self.assertTrue(p.exists())
+        self.assertTrue(p.is_dir())
+        if os.name != 'nt':
+            # the directory's permissions follow the mode argument
+            self.assertEqual(stat.S_IMODE(p.stat().st_mode), 0o555 & mode)
+        # the parent's permissions follow the default process settings
+        self.assertEqual(stat.S_IMODE(p.parent.stat().st_mode), mode)
 
     @with_symlinks
     def test_symlink_to(self):

diff --git a/Misc/NEWS b/Misc/NEWS
index 52b0a3d5c1c989dd4631a45a2eea74d1d497ea1e..3525edf3b44af914a387bcc7a669604022589a26 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -44,6 +44,10 @@ Core and Builtins
 Library
 -------
 
+- Issue #19921: When Path.mkdir() is called with parents=True, any missing
+  parent is created with the default permissions, ignoring the mode argument
+  (mimicking the POSIX "mkdir -p" command).
+
 - Issue #19887: Improve the Path.resolve() algorithm to support certain
   symlink chains.