Skip Menu |

This queue is for tickets about the Exporter-Easy CPAN distribution.

Report information
The Basics
Id: 40987
Status: resolved
Priority: 0/
Queue: Exporter-Easy

People
Owner: NEILB [...] cpan.org
Requestors: nick [...] ccl4.org
Cc:
AdminCc:

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



Subject: INSTALLDIRS=perl should not be set.
Exporter::Easy has INSTALLDIRS=perl set in its Makefile.PL. This indicates to MakeMaker that it is a core module and should be installed with other core modules. Exporter::Easy is not in the core. INSTALLDIRS should not be set. The fact that it is being installed in the core, rather than site_perl, is causing problems for me (and anyone else) who upgrades their perl install to a newer version. (Given the default @INC order). However, I suspect that the fix needs to be careful, as any existing install has the current version of Exporter::Easy installed in the wrong place, but early in @INC ahead of site_perl ( see also http://rt.cpan.org/Public/Bug/Display.html?id=8893 )
Subject: Re: [rt.cpan.org #40987] INSTALLDIRS=perl should not be set.
Date: Mon, 17 Nov 2008 13:42:14 +0000
To: bug-Exporter-Easy [...] rt.cpan.org
From: "Fergal Daly" <fergal [...] esatclear.ie>
Yes unfortunately this is a long standing stupidity that I haven't figured a good way to solve. If you can suggest something better than some attempt to automatically correct past mistakes, I'd be interested. I don't have very much time these days so implementing anything too clever is not going to happen.
Thanks, have removed INSTALLDIRS Neil
Subject: Re: [rt.cpan.org #40987] INSTALLDIRS=perl should not be set.
Date: Sun, 6 Jul 2014 09:58:32 +0900
To: bug-Exporter-Easy [...] rt.cpan.org
From: Fergal Daly <fergald [...] gmail.com>
BTW one of the reasons I left this unfixed is that last time I checked, the perl dir took precedence over the default install door. So people upgrading to s new Verizon would not actually use the upgraded version. If that's still the case, then maybe you need to check for and delete old versions in the perl dir, F On 6 Jul 2014 06:36, "Neil_Bowers via RT" <bug-Exporter-Easy@rt.cpan.org> wrote: Show quoted text
> Queue: Exporter-Easy > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=40987 > > > Thanks, have removed INSTALLDIRS > > Neil > >
Show quoted text
> BTW one of the reasons I left this unfixed is that last time I checked, the > perl dir took precedence over the default install door. So people upgrading > to s new Verizon would not actually use the upgraded version.
Without INSTALLDIRS=perl, the module gets installed in site_perl, which takes precedence over the 'perl' library directory. So if someone already has an older version installed, then installs the current version, when they use it they'll pick up the most recent release. Sorry, I should have noted that I tested this before doing the release. Cheers, Neil
Subject: Re: [rt.cpan.org #40987] INSTALLDIRS=perl should not be set.
Date: Sun, 6 Jul 2014 17:41:58 +0900
To: bug-Exporter-Easy [...] rt.cpan.org
From: Fergal Daly <fergald [...] gmail.com>
The thread is below. At the time, it varied by distribution with Solaris doing 1 thing, Debian another and pure perl maybe something else. I hope recent perls are more reasonable... http://www.gossamer-threads.com/lists/perl/porters/170314?do=post_view_flat#170314 On 6 Jul 2014 17:09, "Neil_Bowers via RT" <bug-Exporter-Easy@rt.cpan.org> wrote: Show quoted text
> Queue: Exporter-Easy > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=40987 > >
> > BTW one of the reasons I left this unfixed is that last time I checked,
> the
> > perl dir took precedence over the default install door. So people
> upgrading
> > to s new Verizon would not actually use the upgraded version.
> > Without INSTALLDIRS=perl, the module gets installed in site_perl, which > takes precedence over the 'perl' library directory. > > So if someone already has an older version installed, then installs the > current version, when they use it they'll pick up the most recent release. > > Sorry, I should have noted that I tested this before doing the release. > > Cheers, > Neil > >