Skip Menu |

This queue is for tickets about the Directory-Scratch CPAN distribution.

Report information
The Basics
Id: 53665
Status: stalled
Priority: 0/
Queue: Directory-Scratch

People
Owner: Nobody in particular
Requestors: groditi [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.14
Fixed in: (no value)



The documentation for randfile states that if the String::Random is avaiable, it will be used. It never states that if String::Random is NOT available, it will die. Try it yourself with "$scratch_dir->randfile(1024);". Either the documentation should be fixed, String::random should be made a dependency, or the method should be changed. As it stands, randfile is useless since Directory::Scratch does not clearly mark the dependency and unexpectedly crashes if you use it and it's not present.
Subject: Re: [rt.cpan.org #53665]
Date: Sat, 16 Jan 2010 02:22:42 -0600
To: bug-Directory-Scratch [...] rt.cpan.org
From: Jonathan Rockway <jrockway [...] cpan.org>
Too late to remove the feature. Just add "use String::Random" before "use Directory::Scratch". But then your program will die if String::Random is not installed, just like it does now. If you have a better way of fixing this problem and not breaking backcompat, please send me a patch. --jrockway