Skip Menu |
 

This queue is for tickets about the Module-Build CPAN distribution.

Report information
The Basics
Id: 54694
Status: resolved
Priority: 0/
Queue: Module-Build

People
Owner: Nobody in particular
Requestors: FIRMICUS [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.3603
Fixed in: (no value)



Subject: cpan_client cannot be found on some systems
Download (untitled) / with headers
text/plain 897b
When I run "./Build installdeps" the cpan_client cannot be found on my Arch Linux system, because perl is located at /usr/bin/perl while cpan is under /usr/bin/perlbin/core/cpan. Module::Build gives the (inaccurate) error message: cpan_client 'cpan' is not executable In fact it was not found at all. The attached patch solves this by adding the config parameters installscript, installsitescript, installvendorscript to the array of dirs to look into. (Actually the "patch" was generated from a bundled Module::Build in a git repo, but it is so trivial that I did not bother). One more thing: I think the lookup order of the dirs should change from install{bin,script}, installsite{bin,script}, installvendor{bin,script} to: installsite{bin,script}, installvendor{bin,script}, install{bin,script} since this follows the order of precedence in Perl's @INC (site > vendor Show quoted text
> core).
Thanks!
Subject: Module-Base.diff
Download Module-Base.diff
text/x-diff 718b
diff --git a/inc/Module-Build/lib/perl5/Module/Build/Base.pm b/inc/Module-Build/lib/perl5/Module/Build/Base.pm index 5bd8ec7..c8ca32d 100644 --- a/inc/Module-Build/lib/perl5/Module/Build/Base.pm +++ b/inc/Module-Build/lib/perl5/Module/Build/Base.pm @@ -3392,6 +3392,7 @@ sub ACTION_installdeps { if ( ! File::Spec->file_name_is_absolute( $command ) ) { my @bindirs = File::Basename::dirname($self->perl); push @bindirs, map {$self->config->{"install${_}bin"}} '','site','vendor'; + push @bindirs, map {$self->config->{"install${_}script"}} '','site','vendor'; for my $d ( @bindirs ) { my $abs_cmd = $self->find_command(File::Spec->catfile( $d, $command )); if ( defined $abs_cmd ) {
Download (untitled) / with headers
text/plain 172b
Thank you for the bug report and patch. It may be a while until it can be applied and released as a number of big projects are occupying the M::B developers at the moment.
Subject: Re: [rt.cpan.org #54694] cpan_client cannot be found on some systems
Date: Wed, 17 Feb 2010 13:11:30 -0800
To: bug-Module-Build [...] rt.cpan.org
From: Eric Wilhelm <enobacon [...] gmail.com>
Download (untitled) / with headers
text/plain 560b
# from François Charette via RT # on Wednesday 17 February 2010 05:50: Show quoted text
>One more thing: I think the lookup order of the dirs should change > from install{bin,script}, installsite{bin,script}, > installvendor{bin,script} to: > installsite{bin,script}, installvendor{bin,script}, > install{bin,script} since this follows the order of precedence in > Perl's @INC (site > vendor > core).
That order actually depends on the perl build, so what we would need to do is detect the @INC order (I don't think the build setting gets into %Config anywhere.) --Eric
patched in repo
Download (untitled) / with headers
text/plain 103b
Now that there has been a stable Module::Build release, I'm marking this "patched" issue as "resolved".


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

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