[PATCH v2] Cygwin: pty: Introduce disable_pcon in environment CYGWIN.

Takashi Yano takashi.yano@nifty.ne.jp
Mon Feb 10 12:25:00 GMT 2020


On Mon, 10 Feb 2020 11:07:10 +0100
Corinna Vinschen wrote:
> 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?

I cannot imagine how to realize this right now. Let me consider.

-- 
Takashi Yano <takashi.yano@nifty.ne.jp>



More information about the Cygwin-patches mailing list