Skip Menu |

This queue is for tickets about the CHI CPAN distribution.

Report information
The Basics
Id: 43409
Status: resolved
Priority: 0/
Queue: CHI

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

Bug Information
Severity: Critical
Broken in: 0.091
Fixed in: 0.28



Subject: cache_root support broken in CHI::Driver::File? root_dir should be used in CHI documentation
In the POD for CHI, the synopsis reads: my $cache = CHI->new( driver => 'File', cache_root => '/path/to/root' ); I'm fairly certain cache_root should be root_dir, as cache_root seems to be the older syntax. As of CHI 0.091, the cache_root syntax seems to be broken. When I put a real directory name in, like this: my $cache = CHI->new( driver => 'File', cache_root => '/real/dirname' ) The object's root dir still ends up being /tmp/chi-driver-file (Perl 5.8.8, Linux). This is bad and confusing; I was trying to figure out how come I had cached data, when my intended cache_root directory was still empty. Whether cache_root is supported or not, as a first step, the docs should be changed to reflect the suggested syntax, since this affects a large proportion of new users. Thank you.
You are correct. I initially intended for cache_root to be a synonym for root_dir, but this never worked, and I no longer consider it necessary. I've fixed the docs. Thanks! On Wed Feb 18 15:45:03 2009, ANIRVAN wrote: Show quoted text
> In the POD for CHI, the synopsis reads: > > my $cache = CHI->new( driver => 'File', cache_root => '/path/to/root' ); > > I'm fairly certain cache_root should be root_dir, as cache_root seems to > be the older syntax. > > As of CHI 0.091, the cache_root syntax seems to be broken. When I put a > real directory name in, like this: > > my $cache = CHI->new( driver => 'File', cache_root => '/real/dirname' ) > > The object's root dir still ends up being /tmp/chi-driver-file (Perl > 5.8.8, Linux). This is bad and confusing; I was trying to figure out how > come I had cached data, when my intended cache_root directory was still > empty. > > Whether cache_root is supported or not, as a first step, the docs should > be changed to reflect the suggested syntax, since this affects a large > proportion of new users. > > Thank you.