Skip Menu |

This queue is for tickets about the Digest-SHA CPAN distribution.

Report information
The Basics
Id: 79815
Status: resolved
Estimated: 1 hour (60 min)
Worked: 1 hour (60 min)
Priority: 0/
Queue: Digest-SHA

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

Bug Information
Severity: Normal
Broken in: 5.71
Fixed in: 5.72



Subject: Install to 'site' instead of 'perl' when perl version is 5.11+
Hello, Starting with version 5.47 (with the patch in RT#34875 from 2008) the Makefile.PL sets INSTALLDIRS to 'perl' so CPAN installs aren't shadowed by the core version. According to [1] this is no longer necessary with perl 5.11 and following, which will look into 'site' first. The attached patch changes the Makefile.PL check from setting INSTALLDIRS to 'perl' when the perl version is at 5.10 or following, to only set it when the perl version is also before 5.11. 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 21:08:24.043937352 +0200 +++ Makefile.PL 2012-09-23 21:09:39.616312097 +0200 @@ -33,7 +33,7 @@ 'DEFINE' => $define, 'INC' => '-I.', 'EXE_FILES' => [ 'shasum' ], - 'INSTALLDIRS' => ($] >= 5.010) ? 'perl' : 'site', + 'INSTALLDIRS' => ($] >= 5.010 and $] < 5.011) ? 'perl' : 'site', ); my $MMversion = $ExtUtils::MakeMaker::VERSION || 0;
On Sun Sep 23 15:18:13 2012, PHAYLON wrote: Show quoted text
> Hello, > > Starting with version 5.47 (with the patch in RT#34875 from 2008) the > Makefile.PL sets INSTALLDIRS to 'perl' so CPAN installs aren't shadowed > by the core version. > > According to [1] this is no longer necessary with perl 5.11 and > following, which will look into 'site' first. > > The attached patch changes the Makefile.PL check from setting > INSTALLDIRS to 'perl' when the perl version is at 5.10 or following, to > only set it when the perl version is also before 5.11. > > regards, > Robert Sedlacek > > [1] https://github.com/rjbs/perlto > do/blob/master/Supply-Patches-to-Fix-Install-Location.mkdn
Thanks for the patch, Robert. I'll publish an updated distribution of Digest::SHA (5.72) sometime within the next several days. Regards, Mark