Message259093
| Author |
martin.panter |
| Recipients |
chris.torek, martin.panter |
| Date |
2016年01月28日.06:08:24 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1453961305.39.0.0623564825524.issue26228@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
I agree with all the changes you made. I made one review comment.
It would be nice to add a test case to expose the problem. Correct me if I am wrong, but it doesn’t look like pty.spawn() is tested at all.
FWIW on Linux, reading from the master end seems to raise EIO if the slave has been closed. And writing to the master when the slave is closed seems to fill up a buffer and eventually blocks.
Ideally I think the best solution for handing exec() failure (Issue 17824) would be to eliminate fork-exec with posix_spawn(); see Issue 20104. But as you say, that’s a separate problem. |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2016年01月28日 06:08:25 | martin.panter | set | recipients:
+ martin.panter, chris.torek |
| 2016年01月28日 06:08:25 | martin.panter | set | messageid: <1453961305.39.0.0623564825524.issue26228@psf.upfronthosting.co.za> |
| 2016年01月28日 06:08:25 | martin.panter | link | issue26228 messages |
| 2016年01月28日 06:08:24 | martin.panter | create |
|