Repository navigation
Conversation
ghost
mentioned this pull request
nicoddemus
left a comment
nicoddemus
left a comment
There was a problem hiding this comment.
Thanks!
Please add a new changelog entry and a test
|
Unfortunately the only place I can reproduce this is on the pyinstaller test suite on py36-win_amd64. For some reason stderr capturing fails resulting in verbose test output and invalid sys.stderr.fileno(). |
|
I see, thanks. Perhaps you can forcibly close the captured file handle inside a test and ensure it doesn't blow up? |
|
Took the liberty of refactoring the test into a subtest. 😁 |
| targetfd_save = self.__dict__.pop("targetfd_save") | ||
| os.dup2(targetfd_save, self.targetfd) | ||
| os.close(targetfd_save) | ||
| # os.close(targetfd_save) |
There was a problem hiding this comment.
Why was this code commented out?
| ef_as_str = pickle.dumps(ef) | ||
| pickle.loads(ef_as_str) | ||
|
|
||
| def test_closed_handle(): |
There was a problem hiding this comment.
I see you overwrote my update to this test, was that deliberate?
There was a problem hiding this comment.
@nicoddemus I am sorry about this; I thought that this had already been merged so I could trash the branch.
|
@nicoddemus I have addressed the root cause of these problems with PyInstaller so I do not think that it is worth the time to try to fix this PR. I am sorry for any wasted time. |
ghost
closed this
|
@xoviat OK then, much thanks anyway for the contribution! 👍 |
|
@xoviat for documentation purposes, can you please link the solution, im curious |
4 participants
works around pytest-dev/pytest-xdist#167. This is not however a "fix" but simply allows the tests to run.