From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 0C6541F4B5; Sat, 16 Nov 2019 11:43:34 +0000 (UTC) Date: Sat, 16 Nov 2019 11:43:34 +0000 From: Eric Wong To: meta@public-inbox.org Subject: Re: [PATCH 29/29] t/common: start_script replaces spawn_listener Message-ID: <20191116114333.GA19173@dcvr> References: <20191115095100.25633-1-e@80x24.org> <20191115095100.25633-30-e@80x24.org> <20191116065231.GA12115@dcvr> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20191116065231.GA12115@dcvr> List-Id: Eric Wong wrote: > "$n" (Net::NNTP) and "$td" (TestDaemon) going out of scope at > the same time seems to trigger some strange ->DESTROY > interaction since both classes have a ->DESTROY method. Nope, I was wrong about that :x > This causes t/v2writable to be stuck until the 60s > EvCleanup::later timer fires (well, I'm pretty sure > it's the EvCleanup::later timer, since it takes ~60s > to fail and not 180s). Actual problem seems to be END {} not firing for EvCleanup.pm because of POSIX::_exit use in the test. But NNTP shutdown seems to have some other problems, too. Will try to sort out sometime later. Anyways, the other 28 patches in this series seem fine and are in master.