[PATCH 0/2] Return appropriate handle by _get_osfhandle() and GetStdHandle().
Corinna Vinschen
corinna-cygwin@cygwin.com
Wed Mar 24 10:52:28 GMT 2021
On Mar 24 11:43, Marco Atzeri wrote:
> On 23.03.2021 11:10, Corinna Vinschen wrote:
> > [CC Marco, CC Jan]
> >
> >
> > > Out of curiosity, I took a quick glance at the cmake code. It appears that
> > > this code is designed to support running cmake in a Console. I don't think
> > > that should be needed any more, if it ever was.
> > > [...]
> > > I think the following might suffice (untested):
> > >
> > > --- a/Source/kwsys/Terminal.c
> > > +++ b/Source/kwsys/Terminal.c
> > > @@ -10,7 +10,7 @@
> > > #endif
> > >
> > > /* Configure support for this platform. */
> > > -#if defined(_WIN32) || defined(__CYGWIN__)
> > > +#if defined(_WIN32)
> > > # define KWSYS_TERMINAL_SUPPORT_CONSOLE
> > > #endif
> > > #if !defined(_WIN32)
>
> noted.
> cmake was always annoying, we remove this type of define and they add
> somewhere else
>
>
> > Looks right to me. If we patch cmake to do the right thing, do we still
> > need this patch, Takashi?
> >
> >
> > Thanks,
> > Corinna
>
> Regards
> Marco
Thanks, Marco!
Corinna
More information about the Cygwin-patches
mailing list