Skip Menu |

This queue is for tickets about the Module-Util CPAN distribution.

Report information
The Basics
Id: 40632
Status: resolved
Priority: 0/
Queue: Module-Util

People
Owner: matt.lawrence [...] virgin.net
Requestors: alexk [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.04
Fixed in: 1.05



Subject: find_in_namespace lost namespace functionality
Hi, when refactoring find_in_namespace() in the latest versions of this module the actual namespace functionality got lost. find_in_namespace() always returns a list of ALL modules in @INC - which supposed to be the "no parameters" fallback behaviour only. The patch attached should fix the issue.
Subject: patch.txt
--- old/Util.pm 2008-11-03 16:35:59.000000000 +0100 +++ new/Util.pm 2008-11-03 16:37:40.000000000 +0100 @@ -194,7 +194,7 @@ for my $root (@inc) { my $ns_root = rel2abs($ns_path, $root); - for my $path (_find_modules($root)) { + for my $path (_find_modules($ns_root)) { my $rel_path = abs2rel($path, rel2abs($root)); push @out, fs_path_to_module($rel_path); }
Subject: Re: [rt.cpan.org #40632] find_in_namespace lost namespace functionality
Date: Mon, 03 Nov 2008 16:25:30 +0000
To: bug-Module-Util [...] rt.cpan.org
From: Matt Lawrence <matt.lawrence [...] virgin.net>
Thanks for that. I'll get this released as soon as I can. It's nice to know someone else is using it! ;-) Matt Alexander Kühne via RT wrote: Show quoted text
> Mon Nov 03 11:16:10 2008: Request 40632 was acted upon. > Transaction: Ticket created by ALEXK > Queue: Module-Util > Subject: find_in_namespace lost namespace functionality > Broken in: 1.04 > Severity: Important > Owner: Nobody > Requestors: alexk@cpan.org > Status: new > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=40632 > > > > Hi, > when refactoring find_in_namespace() in the latest versions of this > module the actual namespace functionality got lost. find_in_namespace() > always returns a list of ALL modules in @INC - which supposed to be the > "no parameters" fallback behaviour only. > > The patch attached should fix the issue. > > >