This queue is for tickets about the Catalyst-Devel CPAN distribution.

Report information
The Basics
Id:
57019
Status:
resolved
Priority:
Low/Low

People
Owner:
bobtfish [...] bobtfish.net
Requestors:
sjn [...] cpan.org
Cc:
AdminCc:

BugTracker
Severity:
Normal
Broken in:
1.27
Fixed in:
(no value)



Subject: File::Copy::Recursive is used in inc/Module/Install/Catalyst.pm
File::Copy::Recursive is used in inc/Module/Install/Catalyst.pm This means that File::Copy::Recursive has to be either specified "configure_requires" in all distros using Module::Install::Catalyst, or that the File::Copy::Recursive::rcopy function is inlined in the same module. You choose. :)
Subject: Re: [rt.cpan.org #57019] File::Copy::Recursive is used in inc/Module/Install/Catalyst.pm
Date: Thu, 29 Apr 2010 02:20:52 +0200
To: "Salve J. Nilsen via RT" <bug-Catalyst-Devel@rt.cpan.org>
From: Florian Ragwitz <rafl@debian.org>
On Wed, Apr 28, 2010 at 08:02:29PM -0400, Salve J. Nilsen via RT wrote:
Show quoted text
> You choose. :)
I'm against the inlining. Another approach would be to have Module;:Install ship the module in inc/, which seems easy enough as it doesn't have any further non-core dependencies. I'd prefer that to a configure requirement. -- BOFH excuse #69: knot in cables caused data stream to become twisted and kinked

Message body not shown because it is not plain text.

On Wed 28. april 2010 20:21:05, FLORA wrote:
Show quoted text
> On Wed, Apr 28, 2010 at 08:02:29PM -0400, Salve J. Nilsen via RT wrote:
> > You choose. :)
> > I'm against the inlining. > > Another approach would be to have Module;:Install ship the module in > inc/, which seems easy enough as it doesn't have any further non-core > dependencies. > > I'd prefer that to a configure requirement.
That would make sense if Module::Install itself used that module, which it doesn't. :-/
Subject: Re: [rt.cpan.org #57019] File::Copy::Recursive is used in inc/Module/Install/Catalyst.pm
Date: Thu, 29 Apr 2010 02:30:35 +0200
To: "Salve J. Nilsen via RT" <bug-Catalyst-Devel@rt.cpan.org>
From: Florian Ragwitz <rafl@debian.org>
On Wed, Apr 28, 2010 at 08:26:27PM -0400, Salve J. Nilsen via RT wrote:
Show quoted text
> That would make sense if Module::Install itself used that module, which > it doesn't. :-/
Module::Install itself doesn't use any Module::Install extensions either. Yet it bundles them when it notices they're being used in a distribution's Makefile.PL. Obviously it doesn't discover any further modules those extensions use, but it provides everything necessary for those extensions to tell it that other modules need bundling as well. -- BOFH excuse #151: Some one needed the powerstrip, so they pulled the switch plug.

Message body not shown because it is not plain text.

I've hit this today with the (emulated) fresh install of perl + cpanm. It should either be specified in configure_requires, or catalyst() call in Makefile.pl under the author mode would call $self->include("File::Copy::Recursive"), so that the module is shipped inside inc/ to CPAN. Either way.
This if fixed in Catalyst::Devel e0fd6615d3, version 1.32. Thanks for the bug report.


This service runs on Request Tracker, is sponsored by The Perl Foundation, and maintained by Best Practical Solutions.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.