Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Dancer CPAN distribution.

Report information
The Basics
Id: 57514
Status: resolved
Priority: 0/
Queue: Dancer

People
Owner: Nobody in particular
Requestors: avar [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 1.176
Fixed in: (no value)



Subject: Dancer leaves behind temporary turds when installing
This is my /tmp after installing Dancer: aoeu tmp (108K) $ ls -l /tmp/ total 80 drwx------ 3 avar avar 4096 2010-05-15 17:31 59YLx9Qtbx drwx------ 3 avar avar 4096 2010-05-15 17:31 ByxsoRaj0n drwx------ 3 avar avar 4096 2010-05-15 17:31 FzQzpcssaX drwx------ 4 avar avar 4096 2010-05-15 17:31 kf36aBfjqu drwx------ 3 avar avar 4096 2010-05-15 17:31 mZIBQb0dvg drwx------ 4 avar avar 4096 2010-05-15 17:31 Sq58GnQnyZ drwx------ 3 avar avar 4096 2010-05-15 17:31 xiAjxMHk5U -rw------- 1 avar avar 6 2010-05-15 17:31 8IQ5lXa4gl -rw------- 1 avar avar 7 2010-05-15 17:31 ADZm2Uytzb -rw------- 1 avar avar 7 2010-05-15 17:31 lhOnJZlYBK -rw------- 1 avar avar 7 2010-05-15 17:31 P3nSnVBahh -rw------- 1 avar avar 7 2010-05-15 17:31 Rxqllgbejd -rw------- 1 avar avar 7 2010-05-15 17:31 zX5nsmddLE You shouldn't use File::Temp in a way that leaves files behind after testing. Instead create a File::Temp tempdir() with CLEANUP => 1 and put the files there. See http://github.com/avar/hailo/blob/master/t/storage/001_meta/switch- order.t#L13 for an example of this.
Show quoted text
> You shouldn't use File::Temp in a way that leaves files behind after > testing. Instead create a File::Temp tempdir() with CLEANUP => 1 and > put the files there.
Hmm, that's weird, we don't use File::Temp at all in Dancer... $ ack 'File::Temp' lib/ $ I'm wondering where these come from...
please provide an example (a test script preferably) to prove that Dancer is responsible for that. I cannot reproduce that bug.

Message body is not shown because it is too large.

FIXED i nthe master branch on GitHub, will be released with the next CPAN upload. http://github.com/sukria/Dancer/commit/5c1ebafd6cf8e3727a06a95d55aa01ac9457f7c3 Thanks for the report.