Skip Menu |

This queue is for tickets about the ExtUtils-MakeMaker CPAN distribution.

Report information
The Basics
Id: 17224
Status: resolved
Priority: 0/
Queue: ExtUtils-MakeMaker

People
Owner: Nobody in particular
Requestors: mcummings [...] gentoo.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in:
  • 6.30
  • 6.30_01
Fixed in: 6.17



Subject: Generated Makefiles reference PERLRUN instead of PERLRUNINST
OK, not sure on affected versions since I just jumped from the core install in 5.8.7 to 5.8.8-rc1 (then bumped to 6.30_01). I noticed this when compiling PDL if that helps any (I might be able to add other packages to the list, that's just the one I've been working on lately). The Makefile that is getting generated defines PERLRUN as the path to perl, PERLRUNINST as PERLRUN+the includes - no problem. But in the generated ProccessPL section (in this example, this case might be relevant to other generated sections not used/needed in PDL) PERLRUN is being used instead of PERLRUNINST, resulting in the build process not being able to see the libs, configs, etc. generated for the local build process.
Subject: Re: [rt.cpan.org #17224] AutoReply: Generated Makefiles reference PERLRUN instead of PERLRUNINST
Date: Mon, 23 Jan 2006 17:05:36 -0500
To: bug-ExtUtils-MakeMaker [...] rt.cpan.org
From: Michael Cummings <mcummings [...] gentoo.org>
The following (pathetic) patch doesn't seem to harm anything (tried against a few other modules) and gets PDL past this error. Not sure what it's worth - this is off of 6.30_01 :) $ diff -Naurp lib/ExtUtils/MM_Unix.pm.orig lib/ExtUtils/MM_Unix.pm --- lib/ExtUtils/MM_Unix.pm.orig 2006-01-23 16:50:40.000000000 -0500 +++ lib/ExtUtils/MM_Unix.pm 2006-01-23 16:51:13.000000000 -0500 @@ -3098,7 +3098,7 @@ sub processPL { my $perlrun; if( defined $self->{PM}{$target} ) { $pm_dep = ''; - $perlrun = 'PERLRUN'; + $perlrun = 'PERLRUNINST'; } else { $pm_dep = 'pm_to_blib';
Download signature.asc
application/pgp-signature 189b

Message body not shown because it is not plain text.

As a side note, PDL can be built with current bleadperl (change number 27902), which hasn't this change (which doesn't feel to be correct anyway to me). But I'm not able to locate where is the fix for this bug between 5.8.8 and bleadperl.
From: mcummings [...] gentoo.org
On Wed Apr 19 09:07:24 2006, RGARCIA wrote: Show quoted text
> As a side note, PDL can be built with current bleadperl (change number > 27902), which hasn't this change (which doesn't feel to be correct > anyway to me). But I'm not able to locate where is the fix for this bug > between 5.8.8 and bleadperl.
I really wish I'd pasted more logs to this ticket. This now works for me in 5.8.8 without my crazy patch following application of the cplusplus patch that came out after 5.8.8 (which is already in blead). I don't see offhand that it was related, but this is the only patch I can think of that I applied to 5.8.8 since filing this. (http://www.gentoo.org/cgi-bin/viewcvs.cgi/dev-lang/perl/files/perl-5.8.8-cplusplus.patch?rev=1.1&view=markup - pulled off a post to p5p in late februrary). I will retest this on a few other boxes with the same set of patches to confirm my install a few minutes ago wasn't a fluke.
Seems to have been a CPANPLUS hiccup.
I've reopened this ticket because I may have found the problem. It may be this code in MM_Unix->process_PL if( defined $self->{PM}{$target} ) { $pm_dep = ''; $perlrun = 'PERLRUN'; } else { $pm_dep = 'pm_to_blib'; $perlrun = 'PERLRUNINST'; } On Windows the files in $self->{PM} might be unix style (since they come from File::Find) but $target might be Windows style so the above if condition might be fooled. I won't have access to a Windows machine until the 17th to check.
From: mcummings [...] gentoo.org
That makes more sense (no cpanplus on this end, so I wasn't sure where you were going with the previous comment).
Ya know, I just noticed this issue has absolutely nothing to do with Windows. The user was using Gentoo with a Perl release candidate and since using the full release Perl has reported no problems. I'm going to chaulk this one up to problems with the 5.8.8 release candiate.