[issue35189] EINTR is not being retried

2018-11-08 Thread Nierob
Nierob added the comment: PR waits for CLA -- keywords: +patch nosy: +nierob pull_requests: +9693 stage: -> patch review ___ Python tracker ___ __

[issue35189] EINTR is not being retried

2018-11-08 Thread STINNER Victor
STINNER Victor added the comment: > https://github.com/nierob/cpython/commit/3b76b887b2b339664dc52c5f246bc7cb268ea616 Oh, nice! Please rebase this change on the master branch and reuse "int async_err = 0;" pattern from Modules/posixmodule.c. You must not raise a new exception if PyErr_CheckS

[issue35189] EINTR is not being retried

2018-11-08 Thread Aapo Samuli Keskimolo
Aapo Samuli Keskimolo added the comment: My colleague has made a prospective fix: https://github.com/nierob/cpython/commit/3b76b887b2b339664dc52c5f246bc7cb268ea616 -- ___ Python tracker

[issue35189] EINTR is not being retried

2018-11-08 Thread STINNER Victor
STINNER Victor added the comment: You're right, it should, but the fcntl module hasn't been patched. Are you interested to work or on patch, or do you want to me to find someone to do it? -- nosy: +vstinner ___ Python tracker

[issue35189] EINTR is not being retried

2018-11-08 Thread Aapo Samuli Keskimolo
New submission from Aapo Samuli Keskimolo : According to https://www.python.org/dev/peps/pep-0475/ the EINTR interruption should be retried automatically, but somehow it does not work and the exception is raised: 2018-11-05 05:21:35,257 ERROR:storage(23491): Remote storage operation failed (