Skip to content

Conversation

@barneygale
Copy link
Contributor

@barneygalebarneygale commented Oct 22, 2024

@barneygale
Copy link
ContributorAuthor

@opk12 how does this look?

@opk12
Copy link

Thank you! Note for the future myself, the call chain is Path.copy() -> Path._copy_file() -> _os.copyfileobj() which does EAFP around FICLONE.

@barneygale
Copy link
ContributorAuthor

Thanks for the reviews, all!

@barneygalebarneygale merged commit ff83499 into python:mainNov 5, 2024
picnixz pushed a commit to picnixz/cpython that referenced this pull request Dec 8, 2024
ebonnal pushed a commit to ebonnal/cpython that referenced this pull request Jan 12, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docsDocumentation in the Doc dirskip newstopic-pathlib

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants

@barneygale@opk12@hugovk@rruuaanng