This is the mail archive of the
cygwin-patches
mailing list for the Cygwin project.
Re: [PATCH 0/1] Cygwin: pty: Make it sure to show up system error messages.
- From: Ken Brown <kbrown at cornell dot edu>
- To: "cygwin-patches at cygwin dot com" <cygwin-patches at cygwin dot com>
- Date: Thu, 5 Sep 2019 16:14:00 +0000
- Subject: Re: [PATCH 0/1] Cygwin: pty: Make it sure to show up system error messages.
- 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=+8caTEUvPRmZo/0i6hxD2kbVxPzlmByydST/+Dl7eKU=; b=n8It+bZ2Cf8gVlTFhTnWhZfiLhiULnfMjekk9OneHQxPKiQIlI9e1F56lJP5WUEdv7RU3JPmXOYvtg08IB3SYERKo5RDoDBXlJQvBvL7OTnehdckM6VqraqJGdIBAT5UIKyR2fXzBwHtEd9fk9KR41ZdZ8MbJsFHGMQoPjXBdcuixbVUlGpQEAn9E9Wtq+LF7LfF+oFEaMDxWWYj5IoU0Kqto2Vux6uq9fshTa1lFmnpMaCqKqqcsVPoSSh3QpvCAdLpm1/uksuc3uunnOuEwBB4R7crTSWKSOQO5gM4xx3K671Z2KEYAaJOrHTjRpx/mflm5Ou+3IeK68OpErGbfQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hD68dYJyKool9/ER8kgFAepwVm3GkV/G/4jIBMFiYHZIWcSGQjxr5KfGON9YRh8h4vqXeN0M6KolDfJRqNRFltVx/7Hrwzaoj6iuVXCQPsxJFHbkrOi27oKHu+4x5/qP3xTb4zroPhqEWn6X7iEB+sccGxz2h9UMl2xLiQ0cXlJgCfhTpL4v12AkNq0cFU/6S4MYE7nz2M9k6tPH+F5f5TW+CQaYBED9kyV69/tvIOCihO4HUEx5X/fcKKUM9d4N9NweumJkefX5129oEpdoPbdKve0Umssrh0TKfIGCPIeYdhjrpTI+1rSL3Y/TUymS0LoLzpIgc5um7nd4Mvi2MA==
- References: <20190905132227.1967-1-takashi.yano@nifty.ne.jp>
On 9/5/2019 9:22 AM, Takashi Yano wrote:
> - Forcibly attach to pseudo console in advance so that the error
> messages by system_printf() is displayed to screen reliably.
> This is needed when stdout is redirected to another pty. In this
> case, process has two ptys opened. However, process can attach
> to only one console. So it is necessary to change console attached.
>
> Takashi Yano (1):
> Cygwin: pty: Make it sure to show up system error messages.
>
> winsup/cygwin/fhandler_tty.cc | 55 +++++++++++++++++++++++++++++++----
> 1 file changed, 49 insertions(+), 6 deletions(-)
Pushed, with a couple of minor typo fixes.
Thanks.
Ken
P.S. Please let me know whenever you think I should make a new test release.
Also, I'll be mostly AFK for about a week starting on Saturday, so I probably
won't be reviewing any patches during that time.