Skip Menu |

This queue is for tickets about the Locale-Codes CPAN distribution.

Report information
The Basics
Id: 54526
Status: resolved
Priority: 0/
Queue: Locale-Codes
People
Owner: Nobody in particular
Requestors: toddr [...] null.net
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 3.00
Fixed in: (no value)

History Show all quoted text
  Thu Feb 11 15:01:30 2010 toddr [...] null.net - Ticket created
Subject: Recent Upgrade breaks install location (installing to site_lib)
The recent 3.0 release of Locale::Codes has changed the install destination from core into site_lib Because Locale::Codes is a perl core module, this is the wrong destination.
  Thu Feb 11 15:07:07 2010 toddr [...] null.net - Correspondence added
From: toddr [...] null.net
I am no Module::Build expert, but I think you need to add this to your Build.PL file based on my theft of how constant (also a dual core module) is doing it: http://cpansearch.perl.org/src/SAPER/constant-1.19/Build.PL installdirs => 'core', then for Makefile.PL do this: INSTALLDIRS => 'perl',
  Thu Feb 11 21:58:27 2010 jdhedden [...] 1979.usna.com - Correspondence added
Subject: [PATCH] Recent Upgrade breaks install location (installing to site_lib)
From: jdhedden [...] 1979.usna.com
On Thu Feb 11 15:07:07 2010, toddr@null.net wrote: Show quoted text
> I am no Module::Build expert, but I think you need to add this to your > Build.PL file based on my > theft of how constant (also a dual core module) is doing it: > http://cpansearch.perl.org/src/SAPER/constant-1.19/Build.PL > > installdirs => 'core', > > then for Makefile.PL do this: > > INSTALLDIRS => 'perl',
The correct setting for this module is: INSTALLDIRS => ((($] >= 5.007003) || ($] < 5.011)) ? 'perl' : 'site'), Patch attached.
Subject: patch.txt
--- Locale-Codes-3.00/Makefile.PL.orig 2010-02-11 21:44:34.191875000 -0500 +++ Locale-Codes-3.00/Makefile.PL 2010-02-11 21:45:07.636750000 -0500 @@ -23,4 +23,5 @@ 'dist' => {COMPRESS=>'gzip',SUFFIX=>'gz'}, PL_FILES => {}, PREREQ_PM => \%requires, + INSTALLDIRS => ((($] >= 5.007003) || ($] < 5.011)) ? 'perl' : 'site'), );
  Thu Feb 11 21:58:28 2010 The RT System itself - Status changed from 'new' to 'open'
  Thu Feb 11 22:17:47 2010 toddr [...] null.net - Correspondence added
From: toddr [...] null.net
On Thu Feb 11 21:58:27 2010, jdhedden@1979.usna.com wrote: Show quoted text
> > The correct setting for this module is: > > INSTALLDIRS => ((($] >= 5.007003) || ($] < 5.011)) ? 'perl' : 'site'), >
This would cause it to be site for > 5.011 My understanding is that only deprecated modules are supposed to behave this way. I've had quite a few conversations this week about this. Did I miss something? If anything it should be this. INSTALLDIRS => (($] >= 5.007003) ? 'perl' : 'site'),
  Fri Feb 12 07:02:04 2010 sbeck [...] cpan.org - Correspondence added
Sorry for the problem. When I assumed maintainership of Locale-Codes, I didn't realize it was a core module. I found it via. CPAN, found it was out-of-date, and set about correcting that. It never dawned on me to check if it was a core module. That said, it appears to me that I should include: Makefile.PL INSTALLDIRS => (($] >= 5.007003) ? 'perl' : 'site') Build.PL installdirs => 'core' but I'm not at all familiar with core module protocol, so I'd like to see agreement from both toddr and jdhedden before committing to that. I'll release a fixed version as soon as consensus is reached. Thanks
  Fri Feb 12 10:13:54 2010 toddr [...] null.net - Correspondence added
From: toddr [...] null.net
I'd encourage you to go to #p5p on irc.perl.org and/or join the p5p mailing list which is the group that maintains perl. At the least, they need to know how to get in touch with you and know when you've made releases so they can include it with perl core. I believe jesse (obra) Is the current pumpking for perl http://lists.cpan.org/showlist.cgi?name=perl5-porters http://use.perl.org/article.pl?sid=09/12/22/199226
  Mon Feb 15 08:15:09 2010 sbeck [...] cpan.org - Correspondence added
I just released 3.01 which installs as a core module. I'll also notify perl5-porters. Thanks
  Mon Feb 15 08:15:23 2010 sbeck [...] cpan.org - Status changed from 'open' to 'resolved'