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



Subject: XSLoader.pm installed in wrong dir
Download (untitled) / with headers
text/plain 396b
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
Download xsloader.patch
text/x-diff 574b
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', },
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>
Download (untitled) / with headers
text/plain 543b
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.
From: JDHEDDEN [...] cpan.org
Seems my change messed up the podcoverage testing. Attached is a patch for it, too.
Download xsloader.patch
text/x-diff 305b
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$']});
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>
Download (untitled) / with headers
text/plain 195b
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.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.