Bug #46055 for CPANPLUS: CPANPLUS::Dist::MM::prepare fails when directory has spaces

Wed May 13 18:36:32 2009 kbrint [...] rufus.net - Ticket created

Subject:

CPANPLUS::Dist::MM::prepare fails when directory has spaces

When CPANPLUS is being installed from a path with spaces, it fails in t/04_CPANPLUS-Module.t. Calling IPC::Cmd::run with a scalar command results in splitting on spaces, which produces the bad results. Instead, since we already have all 4 arguments individually, we should pass an array reference to IPC::Cmd::run(). The attached patch fixes the problem. Please integrate it into a future version of CPANPLUS. Thanks!

Subject:

patch2

Wed May 13 18:38:17 2009 kbrint [...] rufus.net - Correspondence added

I think the patch attachment may not have worked... ? Attaching again.

Download patch2
application/octet-stream 480b

Message body not shown because it is not plain text.

Wed May 13 18:38:18 2009 kbrint [...] rufus.net - Status changed from 'new' to 'open'

Fri Jun 12 11:47:09 2009 kane [...] cpan.org - Correspondence added

On Wed May 13 18:36:32 2009, KBRINT wrote: Show quoted text

> When CPANPLUS is being installed from a path with spaces, it fails in > t/04_CPANPLUS-Module.t. > Calling IPC::Cmd::run with a scalar command results in splitting on > spaces, which produces the > bad results. > > Instead, since we already have all 4 arguments individually, we should > pass an array reference > to IPC::Cmd::run(). > > The attached patch fixes the problem. Please integrate it into a > future version of CPANPLUS.

Thanks, applied with some tweaks.

Fri Jun 12 11:47:12 2009 kane [...] cpan.org - Status changed from 'open' to 'resolved'