Re: [PATCH v7 1/1] Cygwin: pty: add pseudo console support.

[Mark Geisert <mark@maxrnd.com>]

On Thu, 22 Aug 2019, Takashi Yano wrote:
> Hi Corinna, Mark and Thomas,
>
> I worked on the following problem and implemented a solution.
> It is not smart enough, however, it works anyway. This is
> realized by hooking the WIN32 API calls which access console,
> and switching I/O to pseudo console side if console access is
> detected.
>
> I will post it as v8 patch. I'm very happy if you will test.

Hi Takashi,
I've successfully built a Cygwin DLL with your patch but am not able to 
test it appropriately because I'm not running Windows 10 on any machine 
yet (blush!).  I was also unsure how Corinna ran her STC.. whether it was 
under bash+mintty or some other combination.

We may have to wait for somebody else to verify the fix for you.
Sorry for the unhelpful news,

..mark


> On Mon, 19 Aug 2019 15:36:41 +0900
> Takashi Yano wrote:
>> Hi Corinna, Mark and Thomas,
>>
>> On Sat, 17 Aug 2019 11:10:53 -0700 (PDT)
>> Mark Geisert wrote:
>>> On Sat, 17 Aug 2019, Corinna Vinschen wrote:
>>>> On Aug 16 01:09, Takashi Yano wrote:
>>>> It looks like there's some timing problem in terms of stderr output.  I
>>>> just made up the example for the Cygwin ML discussion about the mcmodel.
>>>> When I start the code built with -mcmodel=small I expected the output:
>>>>
>>>> Cygwin runtime failure: /home/corinna/dll/main.exe: Invalid relocation.
>>>> Offset 0xfffffffd80348989 at address 0x40000103b doesn't fit into 32 bits
>>>
>>> That particular error message doesn't go out on Cygwin's stderr.  It's
>>> done by a Windows WriteFile() on STD_ERROR_HANDLE followed by a
>>> FlushFileBuffers() on same.
>>>
>>> Not sure that it matters, but perhaps it does.
[...]