Skip Menu |
 

This queue is for tickets about the Log-Log4perl CPAN distribution.

Report information
The Basics
Id: 34401
Status: resolved
Priority: 0/
Queue: Log-Log4perl

People
Owner: Nobody in particular
Requestors: jgangemi@gmail.com (no email address)
Cc:
AdminCc:

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



Subject: Util::module_available broken for code references
Download (untitled) / with headers
text/plain 186b
Log::Log4perl::Util::module_available does not work properly for code references that may reside in @INC. the attached patch fixes that issue. this may be related to bug 24884 as well.
Subject: Util.diff
Download Util.diff
text/x-diff 482b
Index: Util.pm =================================================================== RCS file: /cvsroot/log4perl/Log-Log4perl/lib/Log/Log4perl/Util.pm,v retrieving revision 1.5 diff -r1.5 Util.pm 29c29 < return 1 if $dir->($dir, $relpath); --- > return 1 if $dir->($dir, $relpath_with_forward_slashes); 31c31 < return 1 if $dir->[0]->($dir, $relpath); --- > return 1 if $dir->[0]->($dir, $relpath_with_forward_slashes);
Download (untitled) / with headers
text/plain 380b
On Mon Mar 24 11:19:00 2008, JAE wrote: Show quoted text
> Log::Log4perl::Util::module_available does not work properly for code > references that may reside in @INC. > > the attached patch fixes that issue. this may be related to bug 24884 as > well.
Are you sure? This would only be different if File::Spec->catfile() uses something other than '/' as a delimiter. Can you provide a test case?
Download (untitled) / with headers
text/plain 622b
On Sun Aug 24 01:53:13 2008, MSCHILLI wrote: Show quoted text
> On Mon Mar 24 11:19:00 2008, JAE wrote:
> > Log::Log4perl::Util::module_available does not work properly for code > > references that may reside in @INC. > > > > the attached patch fixes that issue. this may be related to bug 24884 as > > well.
> > Are you sure? This would only be different if File::Spec->catfile() uses > something other than '/' as a delimiter. Can you provide a test case?
hrm - i'll have to see what i can do about that, we see this b/c we have a coderef as part of @INC, and i'm not 100% sure how to best reproduce that outside of our environment.
Download (untitled) / with headers
text/plain 310b
On Mon Aug 25 10:16:44 2008, JAE wrote: Show quoted text
> hrm - i'll have to see what i can do about that, we see this b/c we have > a coderef as part of @INC, and i'm not 100% sure how to best reproduce > that outside of our environment.
I can see it happening on Windows, but I take it you're running some kind of Unix?
Download (untitled) / with headers
text/plain 467b
On Tue Aug 26 02:02:48 2008, MSCHILLI wrote: Show quoted text
> On Mon Aug 25 10:16:44 2008, JAE wrote: >
> > hrm - i'll have to see what i can do about that, we see this b/c we have > > a coderef as part of @INC, and i'm not 100% sure how to best reproduce > > that outside of our environment.
> > I can see it happening on Windows, but I take it you're running some > kind of Unix?
we run on a bunch of different unix flavors and windows, which is where the issue occurs for us.
Download (untitled) / with headers
text/plain 181b
Sorry for the long wait, finally applied your patch: http://github.com/mschilli/log4perl/commit/14ae276817842ba33818792e96a3b0d3e258fe3b Thanks for your contribution. -- Mike


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.