Sebastian Wiedenroth added the comment:
Excellent analysis, that's it! I've also tested your patch on SmartOS and it
works great.
--
___
Python tracker
<https://bugs.python.o
Change by Sebastian Wiedenroth :
--
keywords: +patch
pull_requests: +22002
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/23085
___
Python tracker
<https://bugs.python.org/issu
New submission from Sebastian Wiedenroth :
I'm investigating some test failures related to sendfile on illumos:
testCount (test.test_socket.SendfileUsingSendfileTest) ... FAIL
testCountSmall (test.test_socket.SendfileUsingSendfileTest) ... ok
testCountWithO
Sebastian Wiedenroth added the comment:
This patch has also been applied to the python versions shipped by OpenIndiana
and OmniOS for a long time. I'd say it's safe to merge.
--
nosy: +wiedi
___
Python tracker
<https://bugs.python.o
Sebastian Wiedenroth added the comment:
> Which Python version do you use?
Myself currently mostly python 3.8 via pkgsrc on SmartOS (one of the illumos
distributions).
OmniOS ships 3.7:
https://github.com/omniosorg/omnios-build/tree/master/build/python37
OpenIndiana ships at least
Sebastian Wiedenroth added the comment:
Please don't.
Even if Oracle doesn't care anymore, the open source illumos community does.
We (illumos) share the same uname "SunOS" and depend on this support.
I have offered to host an illumos buildbot in the past and that offer