Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the XSLoader CPAN distribution.

Report information
The Basics
Id: 22130
Status: resolved
Priority: 0/
Queue: XSLoader
People
Owner: SAPER [...] cpan.org
Requestors: jdhedden [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: 0.07
Fixed in: 0.08

History Show all quoted text
  Mon Oct 16 14:45:15 2006 jdhedden [...] cpan.org - Ticket created
Subject: XSLoader.pm installed in wrong dir
In core Perl, XSLoader.pm is installed under $Config{installarchlib}. However, the CPAN version gets installed under $Config{installprivlib}. Because architecture-based libs are searched first, this actually prevents the new version from being used! (My rationale for 'critical'.) The attached patch corrects this problem so that XSLoader.pm is installed in the same location as in core Perl.
Subject: xsloader.patch
diff -urN XSLoader-0.07/Makefile.PL XSLoader-patched/Makefile.PL --- XSLoader-0.07/Makefile.PL 2006-10-12 19:34:10.000000000 -0400 +++ XSLoader-patched/Makefile.PL 2006-10-16 14:35:30.000000000 -0400 @@ -34,7 +34,7 @@ ABSTRACT_FROM => 'XSLoader_pm.PL', INSTALLDIRS => 'perl', PL_FILES => { 'XSLoader_pm.PL' => 'XSLoader.pm' }, - PM => { 'XSLoader.pm' => '$(INST_LIBDIR)/XSLoader.pm' }, + PM => { 'XSLoader.pm' => '$(INST_ARCHLIB)/XSLoader.pm' }, PREREQ_PM => { 'Test::More' => '0.62', },
  Mon Oct 16 16:42:11 2006 maddingue [...] free.fr - Correspondence added
Subject: Re: [rt.cpan.org #22130] XSLoader.pm installed in wrong dir
Date: Mon, 16 Oct 2006 22:41:46 +0200
To: bug-XSLoader [...] rt.cpan.org
From: Sébastien Aperghis-Tramoni <maddingue [...] free.fr>
Jerry D. Hedden wrote: Show quoted text
> In core Perl, XSLoader.pm is installed under $Config{installarchlib}. > However, the CPAN version gets installed under $Config > {installprivlib}. > Because architecture-based libs are searched first, this actually > prevents the new version from being used! (My rationale for > 'critical'.) > > The attached patch corrects this problem so that XSLoader.pm is > installed in the same location as in core Perl.
Oh, well spotted. Thanks, applied. -- Sébastien Aperghis-Tramoni Close the world, txEn eht nepO.
  Mon Oct 16 16:42:12 2006 The RT System itself - Status changed from 'new' to 'open'
  Mon Oct 16 16:48:57 2006 SAPER [...] cpan.org - Taken
  Mon Oct 16 20:10:12 2006 jdhedden [...] cpan.org - Correspondence added
From: JDHEDDEN [...] cpan.org
Seems my change messed up the podcoverage testing. Attached is a patch for it, too.
Only in XSLoader-patched: XSLoader.pm diff -r XSLoader-0.07/t/podcover.t XSLoader-patched/t/podcover.t 1d0 < #!/usr/bin/perl/perl -T 6c5,7 < all_pod_coverage_ok({also_private => ['^bootstrap_inherit$']}); --- > > plan tests => 1; > pod_coverage_ok('XSLoader', {also_private => ['^bootstrap_inherit$']});
  Tue Oct 17 03:04:55 2006 maddingue [...] free.fr - Correspondence added
Subject: Re: [rt.cpan.org #22130] XSLoader.pm installed in wrong dir
Date: Tue, 17 Oct 2006 09:04:37 +0200
To: bug-XSLoader [...] rt.cpan.org
From: Sébastien Aperghis-Tramoni <maddingue [...] free.fr>
Jerry D. Hedden wrote: Show quoted text
> Seems my change messed up the podcoverage testing. > Attached is a patch for it, too.
Thanks, applied. -- Sébastien Aperghis-Tramoni Close the world, txEn eht nepO.
  Sat Sep 15 21:53:30 2007 SAPER [...] cpan.org - Status changed from 'open' to 'resolved'
  Sat Sep 15 21:53:32 2007 SAPER [...] cpan.org - Fixed in 0.08 added