Skip Menu |

This queue is for tickets about the TAP-Formatter-HTML CPAN distribution.

Report information
The Basics
Id: 37019
Status: resolved
Priority: 0/
Queue: TAP-Formatter-HTML

People
Owner: steve [...] purkis.ca
Requestors: schwigon [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.03
Fixed in: 0.06



Subject: tempdir without cleanup
Hi! The use of tempdir() in TAP::Formatter::HTML creates lots of subdirs in /tmp. It seems that simply using the CLEANUP option can fix that. See patch. Thanks and regards, Steffen -- Steffen Schwigon <ss5@renormalist.net> Dresden Perl Mongers <http://dresden-pm.org/> Deutscher Perl-Workshop <http://www.perl-workshop.de/>
Subject: tempdir_cleanup.patch
--- TAP/Formatter/HTML.pm.orig 2008-06-23 14:06:20.000000000 +0200 +++ TAP/Formatter/HTML.pm 2008-06-23 14:06:36.000000000 +0200 @@ -61,7 +61,7 @@ use constant default_css_uris => ['file:TAP/Formatter/HTML/default_report.css']; use constant default_template_processor => Template->new( - COMPILE_DIR => catdir( tempdir(), 'TAP-Formatter-HTML' ), + COMPILE_DIR => catdir( tempdir( CLEANUP => 1 ), 'TAP-Formatter-HTML' ), COMPILE_EXT => '.ttc', INCLUDE_PATH => join(':', @INC), );
On Mon Jun 23 08:21:13 2008, SCHWIGON wrote: Show quoted text
> Hi! > > The use of tempdir() in TAP::Formatter::HTML creates > lots of subdirs in /tmp. It seems that simply using > the CLEANUP option can fix that. See patch.
Thanks Steffen, I wish I'd seen this before the last release - will apply for 0.06. -Steve
This was fixed in 0.06. Forgot to close it then.