Skip to content
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.

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

bpo-36610: shutil.copyfile(): use sendfile() on Linux only #13675

Merged
merged 2 commits into from
May 30, 2019
Merged

bpo-36610: shutil.copyfile(): use sendfile() on Linux only #13675

merged 2 commits into from
May 30, 2019

Conversation

giampaolo
Copy link
Contributor

@giampaolo giampaolo commented May 30, 2019

...and avoid using it on Solaris as it can raise EINVAL if offset is equal or bigger than the size of the file, see:
https://bugs.python.org/issue36610

https://bugs.python.org/issue36610

...and avoid using it on Solaris as it can raise EINVAL if offset is equal or bigger than the size of the file
@giampaolo giampaolo merged commit 413d955 into python:master May 30, 2019
@giampaolo giampaolo deleted the shutil-cp-sendfile-solaris branch May 30, 2019 06:05
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
…13675)

...and avoid using it on Solaris as it can raise EINVAL if offset is equal or bigger than the size of the file
@stellarpower
Copy link

I have just installed Python 3.10a6 because of this but am still getting this error. I'm inside a Ubuntu LX zone so it's actually coming all the way through into the guest system, I am guessing because the patch is using the OS name and correctly determining the guest to be linux. I'll ask with the Solaris people to see if this is possibly an issue with our LX translation of syscalls.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants