[issue35633] test_eintr fails on AIX since fcntl functions were modified

Change by STINNER Victor : -- nosy: -vstinner ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Kevin added the comment: FYI, the problem here is that AIX fcntl returns EACCES in the case that the lock is held and non-blocking behavior was requested: > The lockfx and lockf subroutines fail if one of the following is true: Item > > EACCESThe Command parameter is F_SETLK, the l_

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Michael Felt added the comment: On 10/03/2019 19:34, Michael Felt wrote: > Michael Felt added the comment: > > Could this also be backported to version 3.6? > > -- > versions: +Python 3.6 Not to worry: As 3.6 is in security mode. > > ___ > Python tra

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Michael Felt added the comment: Could this also be backported to version 3.6? -- versions: +Python 3.6 ___ Python tracker ___ ___ P

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Change by Michael Felt : -- pull_requests: +11940 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue35633] test_eintr fails on AIX since fcntl functions were modified

miss-islington added the comment: New changeset 7e618f3154404613cd9774fb8ffb5b48e7233a9e by Miss Islington (bot) in branch '3.7': bpo-35633: test_lockf() fails with "PermissionError: [Errno 13] Permission denied" on AIX (GH-11424) https://github.com/python/cpython/commit/7e618f3154404613cd97

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Mariatta Wijaya added the comment: Thanks all! -- nosy: +Mariatta resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ _

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Change by miss-islington : -- pull_requests: +11890 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://m

[issue35633] test_eintr fails on AIX since fcntl functions were modified

miss-islington added the comment: New changeset b94d4bed67c60834b1a5a0936b8c13934cf3b872 by Miss Islington (bot) (Michael Felt) in branch 'master': bpo-35633: test_lockf() fails with "PermissionError: [Errno 13] Permission denied" on AIX (GH-11424) https://github.com/python/cpython/commit/b9

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Michael Felt added the comment: Done. > On 1/15/2019 11:23 AM, STINNER Victor wrote: > I would prefer to simply skip the lockf() test rather than ignoring > PermissionError for flock() and lockf() on all platforms. Can you please > write a PR for that? > > There is no need to add a NEWS ent

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Michael Felt added the comment: I’ll make a new PR and delete the news entry. Sent from my iPhone > On 15 Jan 2019, at 11:23, STINNER Victor wrote: > > > STINNER Victor added the comment: > >> On AIX the test for flock() passes, but the test for lockf() fails: (...) > > I would prefer t

[issue35633] test_eintr fails on AIX since fcntl functions were modified

STINNER Victor added the comment: > On AIX the test for flock() passes, but the test for lockf() fails: (...) I would prefer to simply skip the lockf() test rather than ignoring PermissionError for flock() and lockf() on all platforms. Can you please write a PR for that? There is no need to

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Michael Felt added the comment: On 07/01/2019 15:46, STINNER Victor wrote: > STINNER Victor added the comment: > > Since you are getting indentation error, I'm not sure about your test. Can > you please apply the patch below and run again test_eintr? Does it still fail > with PermissionError

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Michael Felt added the comment: On 07/01/2019 15:46, STINNER Victor wrote: > STINNER Victor added the comment: > > Since you are getting indentation error, I'm not sure about your test. Can > you please apply the patch below and run again test_eintr? Does it still fail > with PermissionError?

[issue35633] test_eintr fails on AIX since fcntl functions were modified

STINNER Victor added the comment: Since you are getting indentation error, I'm not sure about your test. Can you please apply the patch below and run again test_eintr? Does it still fail with PermissionError? diff --git a/Lib/test/eintrdata/eintr_tester.py b/Lib/test/eintrdata/eintr_tester.

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Michael Felt added the comment: On 04/01/2019 23:42, STINNER Victor wrote: > STINNER Victor added the comment: > > Does the test pass if you open the file in read+write ("w+b") mode rather > than write-only ("wb") mode? > > I'm talking about this line: > > open(support.TESTFN, 'wb') > > Note:

[issue35633] test_eintr fails on AIX since fcntl functions were modified

STINNER Victor added the comment: Does the test pass if you open the file in read+write ("w+b") mode rather than write-only ("wb") mode? I'm talking about this line: open(support.TESTFN, 'wb') Note: if you want to test, you have the modify the mode twice: "with open('%s', 'wb') as f:" % s

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Change by Michael Felt : -- components: -IO ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyt

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Change by Michael Felt : -- keywords: +patch, patch pull_requests: +10840, 10841 stage: -> patch review ___ Python tracker ___ ___

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Change by Michael Felt : -- keywords: +patch pull_requests: +10840 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-li

[issue35633] test_eintr fails on AIX since fcntl functions were modified

Michael Felt added the comment: After reading the PEP I realized it is much simpler. The test is for interrupts that occur at a low-level - and not for permission issues. The test is failing because there is a permission issue, not a missed interrupt issue. Modifying the code to: (see line 5

[issue35633] test_eintr fails on AIX since fcntl functions were modified

New submission from Michael Felt : test_eintr fails on AIX since fcntl functions were modified In issue35189 the fnctl() module was modified so that the EINTR interruption should be retried automatically. On AIX the test for flock() passes, but the test for lockf() fails: ===