This is the mail archive of the
cygwin-patches
mailing list for the Cygwin project.
Re: [PATCH v2 0/1] Fix PTY state management in pseudo console support.
- From: Ken Brown <kbrown at cornell dot edu>
- To: "cygwin-patches at cygwin dot com" <cygwin-patches at cygwin dot com>
- Date: Mon, 2 Sep 2019 13:10:27 +0000
- Subject: Re: [PATCH v2 0/1] Fix PTY state management in pseudo console support.
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cornell.edu; dmarc=pass action=none header.from=cornell.edu; dkim=pass header.d=cornell.edu; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=QxIzi5Z4js4i6mU1rNfNqSJKVo/noYpk4Depydb6cTQ=; b=XlzNc6o2L4XD6dZSs2ZolDBP2ihstJk7QoMz9jv+H60JDulfHi5Wob4wHNAuOvv+eDXOZzTu98uv14OcWhKiyDOqo3yzqC3wkEIui6sZMbHENpAv6EjN8VeAtNAP1A5bsoWULDoYEWN1w9knJ6Oali0JtFGP2n1iTDlET6brGwoa/w+kbFhtj8jRsD4lkzRt+k65mdfVAp1kpZszCvv3zyGgWJly+f7DFw875vk+rEaxxVFu0BN+Nyh0RGuJp1NZuocUeC9/u+wD6iYX4AGADdGJKrwnjir6eqZuGVmNs+mTLFMBw6OfM4q6eggYDyhyy9uzGaFxbC+U5SeFmhxRiQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=AZ6ImYafSfHLP63bbyJyAr2rmXiQ/1SRxASOLbGoJXDNq2YWA+08PMugD84R42YklirROnHdmHG56+5bX3Cq9FRAb5HT0YiuRlE4PRk6IGgio1hRP0G9NCBnrzANG2D4HN4udvzyf16x8Owf2y+ATPtWEkmXFDHn7Jayy3+nrv835lHscojr2EPOqLPRCMIOUWsgUeebYT4eeH8mj3+T7QgeHFjxIQj5+WDOq/QPvWvrzlcSw56VgYcHmZlBVcBo2WUYE6ifCPVgyASt3l5ptTUGsRPPYoEljrhIeHcRKsUjkpMKm36WpxYUjPvz0HtMvWF3/gkbPNlhZaNSgc+ioA==
- References: <20190831225446.1506-1-takashi.yano@nifty.ne.jp> <1169565b-6e96-2865-4cad-eb7b2e6abe05@cornell.edu> <20190902065321.5c288f415052f88b6392622c@nifty.ne.jp>
On 9/1/2019 5:53 PM, Takashi Yano wrote:
> Hi Ken,
>
> Thank you for testing.
>
> On Sun, 1 Sep 2019 15:13:47 +0000
> Ken Brown wrote:
>> On 8/31/2019 6:54 PM, Takashi Yano wrote:
>>> Pseudo console support in test release TEST: Cygwin 3.1.0-0.3,
>>> introduced by commit 169d65a5774acc76ce3f3feeedcbae7405aa9b57,
>>> has some bugs which cause mismatch between state variables and
>>> real pseudo console state regarding console attaching and r/w
>>> pipe switching. This patch fixes this issue by redesigning the
>>> state management.
>>
>> After applying this patch, I get the following in mintty:
>>
>> $ cygcheck -cd | grep bash
>> grep: write error: Bad file descriptor
>>
>> Further commands after that lead to the cursor jumping around.
>
> I have fixed this problem. I will post it as v3 patch soon.
>
>> Here's a second glitch I've noticed (starting with commit
>> 169d65a5774acc76ce3f3feeedcbae7405aa9b57): In emacs, if I run a command that
>> uses compilation mode, the output displayed in the compilation buffer starts
>> with ^[[H^[[J. Here ^[ is the escape character, so this is apparently the two
>> ANSI escape sequences ESC[H and ESC[J.
>>
>> Sample commands that use compilation mode are 'M-x compile', 'M-x rgrep', and
>> 'M-x find-name-directory'. I can provide more detailed reproduction
>> instructions if you're not an emacs user.
>
> Hmmm, it seems that ANSI escape sequences are not recognized in emacs.
>
>> 'M-x find-name-directory'
> Do you mean find-name-dired ?
Yes.
Ken