[PATCH v2] Cygwin: pty: Introduce disable_pcon in environment CYGWIN.
Corinna Vinschen
corinna-cygwin@cygwin.com
Mon Feb 10 10:07:00 GMT 2020
On Feb 8 18:13, Thomas Wolff wrote:
> 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
Interesting idea, but given that all the Pseudo Console code in
Cygwin is from Takashi, he should decide how to go forward.
Takashi? What do you think?
Thanks,
Corinna
--
Corinna Vinschen
Cygwin Maintainer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://cygwin.com/pipermail/cygwin-patches/attachments/20200210/cc88a295/attachment.sig>
More information about the Cygwin-patches
mailing list