From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM12-MW2-obe.outbound.protection.outlook.com (mail-mw2nam12on2109.outbound.protection.outlook.com [40.107.244.109]) by sourceware.org (Postfix) with ESMTPS id 9B6AA3857024 for ; Wed, 22 Jul 2020 12:13:51 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 9B6AA3857024 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=D/uy6N+Nta8w4N6ocnnOQcXbD0OIB9C+VDJJwstIMwSMswzoLxg2HFrRn5Ce3z64Amp2tkLvbnP2n3FaQ7qgHgsh/6vuH2VMXy9KhfVImNZNitZab7Q+1PWLk5G96Mt51Buk72DcL8PRiCs6n443TT8jfKnPj5ad3DNgvEW8TBLyNDBYpFdy5wbq8+VbVWyAsNCSHu9UINyy2ZhgCjAGJAnLGcKJQhx4rDmqwuI15+ULMysyuo37po0lP9jrYEY4z0fQvNRiy54wZNccT6kVe9j3Mp+tu+x7JGyfEEFt9HvUExqw0UsDkrd3gMVo6+uIJxQB7wPNWJ+1EYKJvkDJRg== 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=HccgRffxQER4xKQwL6wMRYqLB8MHDfjMvDRazjhPLVc=; b=ogvk2iqviSGguc6G73numhFKR6DDXq6hmfHVFi1r5UoIHFaANVysB+fOuza9TZlDamOlOyCBFVHdbzOrlHtVpXX1wbhu4FoWin6r5/9EqjXDDWW97ZDnsYiMrq98+0Du0eg71DIpwbutjZH/tJ5fyEZmyUViUlaeijQjrTGtN5jy5VaJpH1Yn7zaXmKlHfgTv/r4aKo1IO5ltx5m6qpi/R3OUpdHCf1CIoPEAcaNnxvzrut+JFP2y15rAyFMW69k0vsDOU5VNfHB7gWknIi5KahDh+mb6MtXilbXhJ5wzYLT/dJ7+i4iCmXJkhIdgAh4Yx0F0LGhkUDLcIrysgHlcQ== 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 Received: from BN8PR04MB6163.namprd04.prod.outlook.com (2603:10b6:408:5c::27) by BN8PR04MB6161.namprd04.prod.outlook.com (2603:10b6:408:50::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3216.23; Wed, 22 Jul 2020 12:13:48 +0000 Received: from BN8PR04MB6163.namprd04.prod.outlook.com ([fe80::2564:aaf4:2a80:e784]) by BN8PR04MB6163.namprd04.prod.outlook.com ([fe80::2564:aaf4:2a80:e784%6]) with mapi id 15.20.3216.022; Wed, 22 Jul 2020 12:13:48 +0000 Subject: Re: New implementation of pseudo console support (experimental) To: cygwin-developers@cygwin.com References: <20200519224018.746f6324badd8a5762df42b1@nifty.ne.jp> <20200525195332.cf60ab2f38d167a76513740e@nifty.ne.jp> <20200526100955.30b1a2baea517e0565f30db6@nifty.ne.jp> <20200529004024.0c2ac3c68b588bda987e0837@nifty.ne.jp> <20200701204751.27609a8f5701010468521a87@nifty.ne.jp> <20200717111912.GG3784@calimero.vinschen.de> <20200718143024.45920213c7aa81912939d944@nifty.ne.jp> <20200720080613.GE16360@calimero.vinschen.de> <20200722031751.e02bf24a7519b43d80e5db71@nifty.ne.jp> <20200722174541.c8113635236fd217cb9ebb77@nifty.ne.jp> <20200722114942.GU16360@calimero.vinschen.de> From: Ken Brown Message-ID: Date: Wed, 22 Jul 2020 08:13:45 -0400 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 In-Reply-To: <20200722114942.GU16360@calimero.vinschen.de> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-ClientProxiedBy: MN2PR15CA0010.namprd15.prod.outlook.com (2603:10b6:208:1b4::23) To BN8PR04MB6163.namprd04.prod.outlook.com (2603:10b6:408:5c::27) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from [IPv6:2604:6000:b407:7f00:ad12:1782:80ed:dfca] (2604:6000:b407:7f00:ad12:1782:80ed:dfca) by MN2PR15CA0010.namprd15.prod.outlook.com (2603:10b6:208:1b4::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3216.21 via Frontend Transport; Wed, 22 Jul 2020 12:13:47 +0000 X-Originating-IP: [2604:6000:b407:7f00:ad12:1782:80ed:dfca] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 78c1cb9e-bece-44be-53d9-08d82e38b029 X-MS-TrafficTypeDiagnostic: BN8PR04MB6161: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:8273; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: TPEcvlR3vmVm7ul4EyfFAxDQv9Rzq/P8NQ90LNai7njlSD+NUDMe3oe2DJcIEP4lrbamUeOtv+SCG1v2CawJ/2iaIRufb7f/A0msK8YWmFsTbWQrFRcloSuU1VCZzPoK7IEcgXLb4oc+uMJ33sj5uwj3r5tHVuHdSFA5E8CPRHdFmpPqhIpbwC6okpzE04hG4yN/LnxxNmkd/s45wKU/gKPqmSr+tRePogAIMs7kwbYmkC/luj4PknZTeS2rHNYxUiEidFqKlKdeaaAvo36MIKnKm2KsD9D2szTy050k6nQ7cXaRySTUqUnsAqhoyf+Sa7Y8mMXaQEp2Iwa9uGXajlLa5EKurm0l7jPLNR/Y+rDAI0eQt4/UKrXrdqcL+n0n X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BN8PR04MB6163.namprd04.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(4636009)(376002)(346002)(136003)(396003)(39850400004)(366004)(31686004)(6916009)(36756003)(75432002)(5660300002)(52116002)(66556008)(66476007)(66946007)(53546011)(2616005)(86362001)(6486002)(31696002)(316002)(478600001)(786003)(2906002)(8676002)(16526019)(186003)(8936002)(43740500002); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData: 64REFxZ5BjIxi1vcYXRPl1k7HXtync4OnKaqTtYTYhDk2BxXzwHl3LjudSTC0R8igKlsOFhOpV7Mqv+NTLDP0z0/suBe+DGkAzYr8oLhwT5wkuxSTpzSb3zpTQnsWiFfB6f7f4nmReLR3d+qh3ZS6rnjUXh+0M5+8FXUm30v4Le1MHPg4Gz+pX2nQo2m4603341HC8igyzi6WlqGom653jVa49c8uNcGcdKbZcYdgk8bHTyIwql7rmKpQrYp8BpRREiRd9p0R5Pavv/SSgHfNl1Ly9lq9pOCTPm5C8vgLhhmN/Oq8zROK3vfCpQsUM9DKb23SR1NlLiUpbvdRG6uM33Cm2DiLzHvKeO4YIM6mLsXWcXMbYc6m5uYoWVpxhcc6l4hqP4WJ/5yg9mTxVQ3IcTZCXxNFQRqxKNpdTEV1klE3YqeCIly90VN2PrHzav18GGpQyUTyqKKSvpTVm6j/DgW0rDcYgiGLs5CmOJ0+Qy1Cz5cddPNI3nRDU8K72gtFt7gkGEQpAM9uZpwuYxUZx8PF/1rLQVjotK0oUvbXpA= X-OriginatorOrg: cornell.edu X-MS-Exchange-CrossTenant-Network-Message-Id: 78c1cb9e-bece-44be-53d9-08d82e38b029 X-MS-Exchange-CrossTenant-AuthSource: BN8PR04MB6163.namprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Jul 2020 12:13:48.3238 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 5d7e4366-1b9b-45cf-8e79-b14b27df46e1 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 8niccWkRE2dOIEnDFZcUG8BET5vX3Gs1fdkoKgRkqbpP+iOuSsOog2wkFbOhp84B6sKkOUQ+KvIfOEj6ywVupw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN8PR04MB6161 X-Spam-Status: No, score=-2.9 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, JMQ_SPF_NEUTRAL, MSGID_FROM_MTA_HEADER, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2, SPF_HELO_PASS, SPF_PASS, TXREP autolearn=no autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: cygwin-developers@cygwin.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Cygwin core component developers mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 22 Jul 2020 12:13:53 -0000 On 7/22/2020 7:49 AM, Corinna Vinschen wrote: > On Jul 22 17:45, Takashi Yano via Cygwin-developers wrote: >> On Wed, 22 Jul 2020 03:17:51 +0900 >> Takashi Yano via Cygwin-developers wrote: >>> Hi Corinna, >>> >>> On Mon, 20 Jul 2020 10:06:13 +0200 >>> Corinna Vinschen wrote: >>>> On Jul 18 14:30, Takashi Yano via Cygwin-developers wrote: >>>>> Hi Corinna, >>>>> >>>>> On Fri, 17 Jul 2020 13:19:12 +0200 >>>>> Corinna Vinschen wrote: >>>>>> Hi Takashi, >>>>>> >>>>>> On Jul 1 20:47, Takashi Yano via Cygwin-developers wrote: >>>>>>> [...] >>>>>>> Revise the patch to fit the current git head. >>>>>> >>>>>> are you satisfied with the code? If you want to merge it, >>>>>> I'd bump Cygwin to 3.2. >>>>> >>>>> Since this new implementation has both advantages and disadvantages, >>>>> there might be some options. >>>>> >>>>> 1) Default to new implementation and leave the current one as an >>>>> alternative. Switch them using the environment CYGWIN. >>>>> 2) Default to current implementation and add the new one as an >>>>> alternative. Switch them using the environment CYGWIN. >>>>> 3) Adopt only new implementation and throw the current one away. >>>>> >>>>> What do you think? >>>> >>>> Do you really want to maintain twice as much code doing the same stuff >>>> and constantly having to ask users which version of the code they are >>>> running? The maintenance cost outweighs the advantages, IMHO. >>>> Personally I'd go for option 3. >>> >>> Personally, I feel a tinge of sadness to discard the current code, >>> however, your opinion sounds reasonable. >>> >>> I will submit a new patch in which all the codes specific to the >>> current implementation are removed. >> >> Attached is the patch in git format-patch format. >> All the codes specific to the current implementation are removed. >> >> Despite the utmost care, the changes are relatively large, so some >> degradation may exist. >> >> I will appreciate if you could test. > > I built a new DLL with this patch and this looks pretty good to me so > far. I'm a bad tester for this stuff, though, using just basic Cygwin > tools usually. > > At one point we should probably just apply this patch and create > developer snapshots to broaden the tester base, but for now, maybe > I should upload my test DLL somewhere? Is anybody besides Thomas > willing to test this new pseudo console stuff? I don't have any specific tests for the pseudo console, but I'll install Takashi's patch and do some things that have often caught bugs in the past (e.g., running through my FIFO test suite). Ken