On Sun, Jul 19, 2009 at 10:16:23PM -0700, Ryan Niebur wrote: > 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?
Yeah, special-casing it in dh-make-perl is the only good short-term solution I can think of. Patching Module::Corelist for Debian to include CPANPLUS::Config::System doesn't seem right. Longer term, I wonder if we should put together a Module::Corelist::Debian (or Module::Corelist::Distro::*) inheriting from Module::Corelist and containing information about the versions of the Debian perl package and quirks like this. This touches an issue with Debian-specific patches to core modules: the core convention is to always increase $VERSION for any change, but there's no separate namespace for vendors so we can't do that easily. Thus our CPANPLUS::Internals::Constants 0.01 is different from the upstream one. Perhaps this information could go in the prospective Module::Corelist::Debian too... Cheers, -- Niko Tyni nt...@debian.org -- To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
