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
[3.15] backport #10333 #10431
[3.15] backport #10333 #10431
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I missed this one before.
See ocaml#10333 Signed-off-by: Etienne Millon <me@emillon.org>
…docker (ocaml#10333) * fix: sendfile() in docker Signed-off-by: Haoxiang Fei <feihaoxiang2014@gmail.com> * use ssize_t for return value Co-authored-by: Etienne Millon <etienne.millon@gmail.com> Signed-off-by: Haoxiang Fei <tonyfettes@tonyfettes.com> * update test Signed-off-by: Etienne Millon <me@emillon.org> * Add changelog Signed-off-by: Etienne Millon <me@emillon.org> --------- Signed-off-by: Haoxiang Fei <feihaoxiang2014@gmail.com> Signed-off-by: Haoxiang Fei <tonyfettes@tonyfettes.com> Signed-off-by: Etienne Millon <me@emillon.org> Co-authored-by: Etienne Millon <etienne.millon@gmail.com> Co-authored-by: Etienne Millon <me@emillon.org>
Signed-off-by: Etienne Millon <me@emillon.org>
Signed-off-by: Etienne Millon <me@emillon.org>
|
||
(3 indicates that the file size is taken modulo 2**32) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We forgot removing this line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indeed. can you remove it on main?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in #10437
Long_val
for sendfile() parameters to fix file copying in docker (fix: useLong_val
for sendfile() parameters to fix file copying in docker #10333)