[PATCH v2] Cygwin: pty: Introduce disable_pcon in environment CYGWIN.
Thomas Wolff
towo@towo.net
Sat Feb 8 17:13:00 GMT 2020
On 22.01.2020 11:06, Corinna Vinschen wrote:
> On Jan 21 22:25, Takashi Yano wrote:
>> - For programs which does not work properly with pseudo console,
>> disable_pcon in environment CYGWIN is introduced. If disable_pcon
>> is set, pseudo console support is disabled.
> Pushed. I just fixed a missing </para> in the doc text.
>
Sorry I didn't notice this before. I think rather than having to decide
and unconditionally switch on or off, a better approach would be to
automatically enable pseudo console when forking a non-cygwin program
only, or have that as a third option. (I think I had suggested this before.)
It's good we had pseudo console in unconditionally now for a while, as
that apparently helped identifying a bunch of issues, but targetting it
to where it's really needed would further help to avoid future trouble,
including any performance issues as recently reported.
I'm willing to prepare a patch if desired, as I had implemented that
condition already for my earlier "winpty injection" proposal.
Thomas
More information about the Cygwin-patches
mailing list