-=| Ryan Niebur, Sun, Jul 19, 2009 at 10:16:23PM -0700 |=-
> On Tue, Jul 14, 2009 at 04:50:28PM +0200, gregor herrmann wrote:
> > 
> > From perl's changelog:
> > 
> > perl (5.10.0-24) unstable; urgency=low
> > [..]
> >   * Save local versions of CPANPLUS::Config::System into /etc/perl.
> >     (See #533707)
> > 
> > lib/DhMakePerl.pm's is_core_module() uses libmodule-corelist-perl's
> > Module::CoreList which probably doesn't know anything about
> > CPANPLUS::Config::System ...
> > 
> > I'm not sure where to fix this actually ...
> > 
> 
> nothing on CPAN will depend on CPANPLUS::Config::System unless it's
> broken. so I propose this patch. what do others think?
> [...]
> +    next if($_ eq "CPANPLUS::Config::System");

I think the right fix would be in Module::CoreList. Luckily we 
maintain libmodule-corelist-perl and dh-make-perl already depends on 
it. A fix may be propagated upstream in the next point release (and 
end up in perl-modules).

I assume (and this worked so far) that each module in 
/usr/{share,lib}/perl/$version is a core one. If this not true, then 
we just can't do anything but trust Module::CoreList and t/corelist.t 
shall be rewritten to simply check a couple of modules we know are 
core ones instead of going over the directories.

I somehow that Module::CoreList is wrong here, so this better be fixed 
there upstream (and patched in libmodule-correlist-perl until upstream 
releases a fixed dist).

-- 
dam

Attachment: signature.asc
Description: Digital signature

Reply via email to