We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
print_exception
I found weird artifact related to this PR during work on #5366. Minimal repeatable example:
# docker run --rm -it xonsh/xonsh:slim xonsh xonfig # xonsh 0.14.0, python 3.11 $RAISE_SUBPROC_ERROR = False $XONSH_SHOW_TRACEBACK = False from xonsh.tools import print_exception def _f(): try: 1/0 except: print_exception(f"Exception") aliases['f'] = _f f # ZeroDivisionError: division by zero # Exception f # <----------------- Empty! f # ZeroDivisionError: division by zero # Exception
After diving into why the second stderr output is forever empty I see that if I remove this line from print_exception() everything is good:
print_exception()
xonsh/xonsh/tools.py
Lines 1056 to 1058 in 55b341d
If this line is working I see that only on the second run of alias the stderr descriptor raises silent error on flushing:
xonsh/xonsh/procs/proxies.py
Line 565 in 55b341d
Lines 40 to 47 in 55b341d
⬇️ Please click the 👍 reaction instead of leaving a +1 or 👍 comment
+1
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
I found weird artifact related to this PR during work on #5366.
Minimal repeatable example:
After diving into why the second stderr output is forever empty I see that if I remove this line from
print_exception()
everything is good:xonsh/xonsh/tools.py
Lines 1056 to 1058 in 55b341d
If this line is working I see that only on the second run of alias the stderr descriptor raises silent error on flushing:
xonsh/xonsh/procs/proxies.py
Line 565 in 55b341d
xonsh/xonsh/procs/proxies.py
Lines 40 to 47 in 55b341d
For community
⬇️ Please click the 👍 reaction instead of leaving a
+1
or 👍 commentThe text was updated successfully, but these errors were encountered: