https://github.com/python/cpython/commit/5b6635f772d187d6049a56bfea76855644cd4ca1
commit: 5b6635f772d187d6049a56bfea76855644cd4ca1
branch: main
author: Barney Gale <[email protected]>
committer: barneygale <[email protected]>
date: 2024-12-06T18:10:00Z
summary:
GH-127381: pathlib ABCs: remove `PathBase.rename()` and `replace()` (#127658)
These methods are obviated by `PathBase.move()`, which can move directories
and supports any `PathBase` object as a target.
files:
M Lib/pathlib/_abc.py
M Lib/pathlib/_local.py
M Lib/test/test_pathlib/test_pathlib_abc.py
diff --git a/Lib/pathlib/_abc.py b/Lib/pathlib/_abc.py
index 86617ff2616f33..11a11ecc4c8203 100644
--- a/Lib/pathlib/_abc.py
+++ b/Lib/pathlib/_abc.py
@@ -14,7 +14,7 @@
import functools
import operator
import posixpath
-from errno import EINVAL, EXDEV
+from errno import EINVAL
from glob import _GlobberBase, _no_recurse_symlinks
from stat import S_ISDIR, S_ISLNK, S_ISREG, S_ISSOCK, S_ISBLK, S_ISCHR,
S_ISFIFO
from pathlib._os import copyfileobj
@@ -902,45 +902,10 @@ def copy_into(self, target_dir, *, follow_symlinks=True,
dirs_exist_ok=dirs_exist_ok,
preserve_metadata=preserve_metadata)
- def rename(self, target):
- """
- Rename this path to the target path.
-
- The target path may be absolute or relative. Relative paths are
- interpreted relative to the current working directory, *not* the
- directory of the Path object.
-
- Returns the new Path instance pointing to the target path.
- """
- raise UnsupportedOperation(self._unsupported_msg('rename()'))
-
- def replace(self, target):
- """
- Rename this path to the target path, overwriting if that path exists.
-
- The target path may be absolute or relative. Relative paths are
- interpreted relative to the current working directory, *not* the
- directory of the Path object.
-
- Returns the new Path instance pointing to the target path.
- """
- raise UnsupportedOperation(self._unsupported_msg('replace()'))
-
def move(self, target):
"""
Recursively move this file or directory tree to the given destination.
"""
- self._ensure_different_file(target)
- try:
- return self.replace(target)
- except UnsupportedOperation:
- pass
- except TypeError:
- if not isinstance(target, PathBase):
- raise
- except OSError as err:
- if err.errno != EXDEV:
- raise
target = self.copy(target, follow_symlinks=False,
preserve_metadata=True)
self._delete()
return target
diff --git a/Lib/pathlib/_local.py b/Lib/pathlib/_local.py
index bb8a252c0e94e2..250bc12956f5bc 100644
--- a/Lib/pathlib/_local.py
+++ b/Lib/pathlib/_local.py
@@ -4,6 +4,7 @@
import os
import posixpath
import sys
+from errno import EXDEV
from glob import _StringGlobber
from itertools import chain
from _collections_abc import Sequence
@@ -876,6 +877,22 @@ def replace(self, target):
os.replace(self, target)
return self.with_segments(target)
+ def move(self, target):
+ """
+ Recursively move this file or directory tree to the given destination.
+ """
+ self._ensure_different_file(target)
+ try:
+ return self.replace(target)
+ except TypeError:
+ if not isinstance(target, PathBase):
+ raise
+ except OSError as err:
+ if err.errno != EXDEV:
+ raise
+ # Fall back to copy+delete.
+ return PathBase.move(self, target)
+
if hasattr(os, "symlink"):
def symlink_to(self, target, target_is_directory=False):
"""
diff --git a/Lib/test/test_pathlib/test_pathlib_abc.py
b/Lib/test/test_pathlib/test_pathlib_abc.py
index 7ba3fa823a30b9..00153e3f5e997e 100644
--- a/Lib/test/test_pathlib/test_pathlib_abc.py
+++ b/Lib/test/test_pathlib/test_pathlib_abc.py
@@ -1376,8 +1376,6 @@ def test_unsupported_operation(self):
self.assertRaises(e, p.hardlink_to, 'foo')
self.assertRaises(e, p.mkdir)
self.assertRaises(e, p.touch)
- self.assertRaises(e, p.rename, 'foo')
- self.assertRaises(e, p.replace, 'foo')
self.assertRaises(e, p.chmod, 0o755)
self.assertRaises(e, p.lchmod, 0o755)
self.assertRaises(e, p.unlink)
_______________________________________________
Python-checkins mailing list -- [email protected]
To unsubscribe send an email to [email protected]
https://mail.python.org/mailman3/lists/python-checkins.python.org/
Member address: [email protected]
