Visitar URL original
gh-93180: Update documentation of `os.copy_file_range` by illia-v · Pull Request #93182 · python/cpython · GitHub
Skip to content

gh-93180: Update documentation of os.copy_file_range #93182

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
vstinner merged 10 commits into from
Jun 8, 2022
Merged

gh-93180: Update documentation of os.copy_file_range #93182

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
c57e76f
Update docs about an old constraint of `os.copy_file_range`
illia-v May 24, 2022
23bbe3a
Add a note about a known issue of `os.copy_file_range`
illia-v May 24, 2022
1e38126
Document some of optimizations `os.copy_file_range` can use implicitly
illia-v May 24, 2022
caeb624
Add a news entry
illia-v May 24, 2022
59561bf
Update the note
illia-v May 30, 2022
bcb339d
Merge branch 'main' into fix-issue-93180
illia-v May 30, 2022
069651d
Explain what copying as if files are opened as binary means
illia-v Jun 6, 2022
f6152e4
Merge branch 'main' into fix-issue-93180
illia-v Jun 7, 2022
94ef9cc
Remove the news entry
illia-v Jun 7, 2022
7ab57e1
Apply Victor's suggestions
illia-v Jun 7, 2022
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
Update the note
  • Loading branch information
@illia-v
illia-v committed May 30, 2022
commit 59561bf89eb8f61a5f4334d942b079bca01c3f42
9 changes: 4 additions & 5 deletions Doc/library/os.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -817,11 +817,10 @@ as internal buffering of data.

.. note::

This function should not be used for copying a file from special
filesystems like procfs and sysfs because of
`a known issue <https://lore.kernel.org/linux-fsdevel/20210126135012.1.If45b7cdc3ff707bc1efa17f5366057d60603c45f@changeid/>`_.
It will copy nothing and return 0 because file sizes there are not known
to the system.
:func:`os.copy_file_range` should not be used for copying a range of a
Comment thread
illia-v marked this conversation as resolved.
Outdated
pseudo file from a special filesystem like procfs and sysfs. It will
always copy no bytes and return 0 as if the file was empty because of a
known Linux kernel issue.

.. availability:: Linux kernel >= 4.5 or glibc >= 2.27.

Expand Down