Andy Kipp
Andy Kipp
This change in `ProcProxyThread.run` completely solves the issue for simple `echo 1` case: ```xsh sp_stdout = io.TextIOWrapper( #open(self.c2pwrite, "wb", -1) self.stdout , encoding=enc, errors=err ) #... sp_stderr = io.TextIOWrapper( #open(self.errwrite,...
After second iteration of tracing this I see two points: 1. I totally confirm that reducing [second open](https://github.com/xonsh/xonsh/issues/5631#issuecomment-2252347565) the same file descriptor solves the issue. Here is the experiment with...
> 2. After switching to single open and named pipe I see the error: ... > This means that switching to named pipe do not solve the issue with closing...
After experiments I see that we must prevent second open of descriptor. I made this in the drafts (5636 5640) that mentioned above. Switching from os.pipe to named pipe requires...
### ➕ sub issue #5648 I've found additional issue. There is the case when: 1. we're using callable alias and captured object with redirect 2. the process ended and closed...
### ➕ sub issue #5649 I've found another additional issue. There is the case when: 1. we're using callable alias 2. the stdout redirected to stderr inside the operator We...
I've finalized the fix in https://github.com/xonsh/xonsh/pull/5645
After some tests I see that #5645 is not full solution. It reduces simple linear case (but it's 90% of ExecAlias usages). To prove that the fix is not complete...
Hey! PR is welcome!
Hey! Wow! First time seeing this :) 1. Please tell us more about the terminal app and it's version. 2. What if you try the same in iTerm2 or another...