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
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Type: text/plain
Charset: utf8
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 797
Download (untitled) / with headers
text/plain 797b
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.
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Charset: utf8
Content-Type: text/plain
Message-ID: <rt-3.6.HEAD-30017-1251758401-1838.43409-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 1062
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.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.