Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

Report information
The Basics
Id: 79814
Status: resolved
Priority: 0/
Queue: Devel-PPPort

People
Owner: MHX [...] cpan.org
Requestors: rs [...] 474.at
Cc:
AdminCc:

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



Subject: Install to 'site' instead of 'perl' when perl version is 5.11+
Hello, Version 3.0 (see RT#7292 from 2004) changed the Makefile.PL to set INSTALLDIRS to 'perl' for perl versions above 5.7.3 (where Devel-PPPort went into core). According to [1] this is no longer necessary for perl versions above 5.11, which will look in 'site' first. The attached patch amends the Makefile.PL to take the above into account when setting INSTALLDIRS. regards, Robert Sedlacek [1] https://github.com/rjbs/perlto do/blob/master/Supply-Patches-to-Fix-Install-Location.mkdn
Subject: install_to_site_post_5011.patch
--- Makefile.PL.orig 2012-09-23 20:53:57.231639068 +0200 +++ Makefile.PL 2012-09-23 20:56:07.732286188 +0200 @@ -72,7 +72,12 @@ } else { # Devel::PPPort is in the core since 5.7.3 - push @moreopts, INSTALLDIRS => ($] >= 5.007003 ? 'perl' : 'site'); + # 5.11.0+ has site before perl + push @moreopts, INSTALLDIRS => ( + ($] >= 5.007003 and $] < 5.011) + ? 'perl' + : 'site', + ); } if ($opt{'apicheck'}) {
On Sun Sep 23 15:03:59 2012, PHAYLON wrote: Show quoted text
> Hello, > > Version 3.0 (see RT#7292 from 2004) changed the Makefile.PL to set > INSTALLDIRS to 'perl' for perl versions above 5.7.3 (where Devel-PPPort > went into core). According to [1] this is no longer necessary for perl > versions above 5.11, which will look in 'site' first. > > The attached patch amends the Makefile.PL to take the above into account > when setting INSTALLDIRS. > > regards, > Robert Sedlacek > > [1] https://github.com/rjbs/perlto > do/blob/master/Supply-Patches-to-Fix-Install-Location.mkdn
Thanks, applied. Will be part of the next release. Marcus
This is part of the 3.21 release that just hit CPAN. Marcus