Skip to content

GH-73991: Rework pathlib.Path.copytree() into copy() #122369

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
barneygale merged 20 commits into from
Aug 11, 2024
Merged

GH-73991: Rework pathlib.Path.copytree() into copy() #122369

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
b7ce838
GH-73991: Rework `pathlib.Path.copytree()` into `copy()`.
barneygale Jul 27, 2024
76cf893
Simplify handling of symlinks
barneygale Jul 27, 2024
b10a3b5
Inline _copy_metadata()
barneygale Jul 28, 2024
a67f5a7
Further symlink tweaks
barneygale Jul 28, 2024
017dd86
Fix copying metadata to non-symlinks
barneygale Jul 28, 2024
fcc8af1
Specify target_is_directory when symlinking.
barneygale Jul 28, 2024
57efe1f
Tiny simplification.
barneygale Jul 28, 2024
8bef16b
Skip unnecessary is_dir() when copying symlinks on POSIX.
barneygale Jul 29, 2024
a51ce63
Docs tweaks
barneygale Jul 30, 2024
51b1e6d
Fix CopyFile2 check
barneygale Aug 4, 2024
429c7b6
Merge branch 'main' into merge-copy-copytree
barneygale Aug 4, 2024
023b3a4
Docs tweak
barneygale Aug 4, 2024
4d6d2dc
Address review feedback
barneygale Aug 4, 2024
7801edf
Merge branch 'main' into merge-copy-copytree
barneygale Aug 6, 2024
5ce229b
Merge branch 'main' into merge-copy-copytree
barneygale Aug 7, 2024
5a94525
Return the target as a Path object.
barneygale Aug 7, 2024
3b6405b
Apply suggestions from code review
barneygale Aug 7, 2024
03f9b7f
Address some review feedback
barneygale Aug 7, 2024
02e9237
Simplify tracebacks
barneygale Aug 7, 2024
fed93fc
Use absolute imports
barneygale Aug 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Apply suggestions from code review 
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
  • Loading branch information
@barneygale @AA-Turner
barneygale and AA-Turner authored Aug 7, 2024
commit 3b6405bbc1444b483e2c5cb44d42a9e19fca3607
4 changes: 2 additions & 2 deletions Doc/library/pathlib.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1560,8 +1560,8 @@ Copying, renaming and deleting
and file data are guaranteed to be copied. Set *preserve_metadata* to true
to ensure that file and directory permissions, flags, last access and
modification times, and extended attributes are copied where supported.
This argument has no effect when copying files on Windows (metadata is
always preserved in this case.)
This argument has no effect when copying files on Windows (where
metadata is always preserved).

If *ignore* is given, it should be a callable accepting one argument: a
source file or directory path. The callable may return true to suppress
Expand Down