Bug #57019 for Catalyst-Devel: File::Copy::Recursive is used in inc/Module/Install/Catalyst.pm

Wed Apr 28 20:02:28 2010 sjn [...] cpan.org - Ticket created

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. :)

Wed Apr 28 20:21:05 2010 rafl [...] debian.org - Correspondence added

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

Download signature.asc
application/pgp-signature 189b

Message body not shown because it is not plain text.

Wed Apr 28 20:21:07 2010 The RT System itself - Status changed from 'new' to 'open'

Wed Apr 28 20:26:27 2010 sjn [...] cpan.org - Correspondence added

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. :-/

Wed Apr 28 20:30:48 2010 rafl [...] debian.org - Correspondence added

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.

Download signature.asc
application/pgp-signature 189b

Message body not shown because it is not plain text.

Fri Mar 04 21:13:58 2011 MIYAGAWA [...] cpan.org - Correspondence added

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.

Tue Mar 22 05:15:54 2011 bobtfish [...] bobtfish.net - Taken

Tue Mar 22 05:21:19 2011 bobtfish [...] bobtfish.net - Correspondence added

This if fixed in Catalyst::Devel e0fd6615d3, version 1.32. Thanks for the bug report.

Tue Mar 22 05:21:20 2011 bobtfish [...] bobtfish.net - Status changed from 'open' to 'resolved'