Bug #32995 for Config-Any:

Thu Feb 07 13:02:21 2008 peter [...] dragonstaff.co.uk - Ticket created

An invalid Perl configuration file causes an error on eval in Config::Any::Perl::load(). However, when running within a test harness like Test::Builder, this error is not fatal and an undef config is returned back to Config::Any. load() should call die in this case and then Config::Any handles it properly. A patch follows: --- Perl.pm 2008-02-07 17:53:23.000000000 +0000 +++ /usr/lib/perl5/site_perl/5.8.5/Config/Any/Perl.pm 2008-02-07 17:52:47.000000000 +0000 @@ -48,6 +48,7 @@ unless ( $content = $cache{ $file } ) { $content = eval { require $file }; + die "eval failed: $file: $@" if $@; $cache{ $file } = $content; }

Thu Feb 07 18:01:07 2008 bricas [...] cpan.org - Taken

Thu Feb 07 18:03:43 2008 bricas [...] cpan.org - Correspondence added

I've checked in a fix, which basically just removes the eval { }, as revision 7438 [1]. Please try it out. -Brian [1] http://dev.catalyst.perl.org/svnweb/Catalyst/revision/?rev=7438

Thu Feb 07 18:03:52 2008 The RT System itself - Status changed from 'new' to 'open'

Fri Feb 08 06:37:59 2008 peter [...] dragonstaff.co.uk - Correspondence added

From:

peterdragon [...] users.sourceforge.net

On Thu Feb 07 18:03:43 2008, BRICAS wrote: Show quoted text

> I've checked in a fix, which basically just removes the eval { }, as > revision 7438 [1]. Please try it out. > > -Brian > > [1] http://dev.catalyst.perl.org/svnweb/Catalyst/revision/?rev=7438

Brian, that works fine for me. Removing the eval was a lot smarter than adding an extra die ;) Regards, Peter

Mon Apr 07 14:25:13 2008 bricas [...] cpan.org - Correspondence added

Version 0.12 now heading to CPAN.

Mon Apr 07 14:25:14 2008 bricas [...] cpan.org - Status changed from 'open' to 'resolved'