-
-
Notifications
You must be signed in to change notification settings - Fork 518
New issue
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
tox 4: support isatty on stdout/stderr #1773
Labels
Comments
gaborbernat
added a commit
that referenced
this issue
Jan 8, 2021
For now sacrificed isatty support on subprocess-es, see #1773 for more details. Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
gaborbernat
added a commit
that referenced
this issue
Jan 8, 2021
This allows users to force isatty until we fix #1773. Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
gaborbernat
added a commit
that referenced
this issue
Jan 8, 2021
This allows users to force isatty until we fix #1773. Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
gaborbernat
changed the title
tox 4 - support isatty on stdout/stderr
tox 4: support isatty on stdout/stderr
Jan 9, 2021
@Julian suggested to take a look at https://github.com/prompt-toolkit/python-prompt-toolkit and https://github.com/urwid/urwid too for inspiration |
This was referenced Feb 2, 2022
blairconrad
added a commit
to blairconrad/dicognito
that referenced
this issue
Dec 8, 2022
masenf
added a commit
to masenf/tox
that referenced
this issue
Dec 14, 2022
The purpose of allocating a pty is to allow better interactive use and enable commands to detect that they are running interactively. From starter code on issue tox-dev#1773
masenf
added a commit
to masenf/tox
that referenced
this issue
Dec 16, 2022
The purpose of allocating a pty is to allow better interactive use and enable commands to detect that they are running interactively. Based on starter code on issue tox-dev#1773. Fixes issue tox-dev#1773.
masenf
added a commit
to masenf/tox
that referenced
this issue
Dec 16, 2022
The purpose of allocating a pty is to allow better interactive use and enable commands to detect that they are running interactively. Based on starter code on issue tox-dev#1773. Fixes issue tox-dev#1773.
This is now fixed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Why? mypy and pytest both use this flag to enable color output support.
Instead of using
subprocess.PIPE
we could pass in the openpty child handler (which would also allow directly setting the termios information on the stream), but once done that the stdin echo seems to no longer buffer realtime, and is instead only line-buffered. This is bad for interacting with the GDB as you can't see what you're typing. Need to understand why, and fix it (if possible).Code for
LocalSubProcessExecuteInstance.get_stream_file_no
:Code to test with:
Of interest links:
The text was updated successfully, but these errors were encountered: