Bug #50184 for Proc-PID-File: [PATCH] test.pl should wait for daemon to exit

Sat Oct 03 20:18:43 2009 jdhedden [...] cpan.org - Ticket created

Subject:

[PATCH] test.pl should wait for daemon to exit

test.pl exits before its test daemon has finished. This causes any immediate subsequent action to cleanup the build dir to fail: rm: cannot remove directory `/var/perl/cpan/build/Proc-PID-File-1.26': Device or resource busy The attached patch fixes this.

Subject:

patch.txt

--- Proc-PID-File-1.26/test.pl.orig 2009-10-03 20:13:10.676083000 -0400 +++ Proc-PID-File-1.26/test.pl 2009-10-03 20:14:01.434852600 -0400 @@ -130,3 +130,7 @@ $c1->release(); ok(! -f $c1->{path}, "oo: released"); + +# - wait for daemon to exit -------------------------------------------------- +sleep 1 while kill 0, $pid; +exit 0;

Sat Oct 03 22:08:56 2009 e [...] arix.com - Correspondence added

Subject:	Re: [rt.cpan.org #50184] [PATCH] test.pl should wait for daemon to exit
Date:	Sat, 03 Oct 2009 19:08:20 -0700
To:	<bug-Proc-PID-File [...] rt.cpan.org>
From:	Erick Calder <e [...] arix.com>

patch applied with many thanks to be included in the upcoming .26 release (this Sunday) On Sat, 03 Oct 2009 20:18:43 -0400, "Jerry D. Hedden via RT" <bug-Proc-PID-File@rt.cpan.org> wrote: Show quoted text

> Sat Oct 03 20:18:43 2009: Request 50184 was acted upon. > Transaction: Ticket created by JDHEDDEN > Queue: Proc-PID-File > Subject: [PATCH] test.pl should wait for daemon to exit > Broken in: 1.26 > Severity: Important > Owner: Nobody > Requestors: jdhedden@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=50184 > > > > test.pl exits before its test daemon has finished. This causes any > immediate subsequent action to cleanup the build dir to fail: > > rm: cannot remove directory `/var/perl/cpan/build/Proc-PID-File-1.26': > Device or resource busy > > The attached patch fixes this.

Sat Oct 03 22:08:57 2009 The RT System itself - Status changed from 'new' to 'open'