Bug #48557 for Catalyst-Plugin-ConfigLoader: Don't localize %ENV

Sun Aug 09 08:30:11 2009 ron [...] rblasch.org - Ticket created

Subject:

Don't localize %ENV

See RT#48555. t/20-mock_load.t and t/21-mock_load_env.t localize %ENV. I don't know about other platforms, but Windows depends quite heavily on some environment variables, like 'Path' for loading libraries. Besides, few will work with a completely empty environment, so it's not really a good test if Catalyst works on the current system. With Catalyst-Plugin-ConfigLoader-0.25, I get error messages like: "...lib/auto/Win32/Win32.dll' for module Win32: load_file:The referenced assembly is not installed on your system" Removing the two localizations makes the tests pass for me again. All tests successful. Files=8, Tests=34, 10 wallclock secs ( 0.03 usr + 0.06 sys = 0.09 CPU) Result: PASS

Mon Aug 10 07:35:12 2009 bricas [...] cpan.org - Correspondence added

Checked in a fix at svn revision 11103 similar to one checked in for Catalyst::Runtime. Please test trunk: http://dev.catalystframework.org/repos/Catalyst/trunk/Catalyst-Plugin-ConfigLoader/

Mon Aug 10 07:35:13 2009 The RT System itself - Status changed from 'new' to 'open'

Mon Aug 10 08:17:44 2009 ron [...] rblasch.org - Correspondence added

On Mon Aug 10 07:35:12 2009, BRICAS wrote: Show quoted text

> Checked in a fix at svn revision 11103 similar to one checked in for > Catalyst::Runtime. > > Please test trunk: > http://dev.catalystframework.org/repos/Catalyst/trunk/Catalyst-Plugin- > ConfigLoader/

Tested r11103 on Windows XP SP3, Perl 5.10. All tests successful. Files=8, Tests=35, 25 wallclock secs ( 0.06 usr + 0.16 sys = 0.22 CPU) Result: PASS Thanks a lot for the fix! Ron

Mon Aug 10 08:34:18 2009 bricas [...] cpan.org - Correspondence added

Thanks for testing. Cheers.

Mon Aug 10 08:34:19 2009 bricas [...] cpan.org - Status changed from 'open' to 'resolved'