Skip Menu |

This queue is for tickets about the Encode CPAN distribution.

Report information
The Basics
Id: 79794
Status: resolved
Priority: 0/
Queue: Encode

People
Owner: Nobody in particular
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+
According to https://github.com/rjbs/perltodo/blob/master/Supply-Patches-to-Fix-Install-Location.mkdn since perl 5.11 distributions can and should install into 'site', as that is the place where perl will look first form that version forward. The attached patch sets INSTALLDIRS to 'site' instead of 'perl' when the perl version is 5.11+. regards, Robert Sedlacek
Subject: install_to_site_post_5011.patch
--- Makefile.PL.orig 2012-09-22 22:41:48.283173479 +0200 +++ Makefile.PL 2012-09-22 22:46:33.604588312 +0200 @@ -45,7 +45,7 @@ INC => '-I' . File::Spec->catfile( '.', 'Encode' ), LICENSE => 'perl', PMLIBDIRS => \@pmlibdirs, - INSTALLDIRS => 'perl', + INSTALLDIRS => ($] < 5.011 ? 'perl' : 'site'), ); package MY;
Thank you, applied accordingly now Dan the Maintainer Thereof On Sat Sep 22 16:55:02 2012, PHAYLON wrote: Show quoted text
> According to > > > https://github.com/rjbs/perltodo/blob/master/Supply-Patches-to-Fix- > Install-Location.mkdn > > since perl 5.11 distributions can and should install into 'site', as > that is the place where perl will look first form that version > forward. > > The attached patch sets INSTALLDIRS to 'site' instead of 'perl' when > the > perl version is 5.11+. > > regards, > Robert Sedlacek