Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Storable CPAN distribution.

Report information
The Basics
Id: 28537
Status: open
Priority: 0/
Queue: Storable

People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 2.16
Fixed in: (no value)



Subject: PATCH for croak.t test failure
The attached patch solves a possible failure in the croak.t test. It seems that in some version Config.pm pre-loads Carp.pm, causing the test to fail. The patch just makes sure that Config.pm is only loaded when building Storable within core perl. Regards, Slaven
Subject: Storable-croak.t.patch
# # # To apply this patch: # STEP 1: Chdir to the source directory. # STEP 2: Run the 'applypatch' program with this patch file as input. # # If you do not have 'applypatch', it is part of the 'makepatch' package # that you can fetch from the Comprehensive Perl Archive Network: # http://www.perl.com/CPAN/authors/Johan_Vromans/makepatch-x.y.tar.gz # In the above URL, 'x' should be 2 or higher. # # To apply this patch without the use of 'applypatch': # STEP 1: Chdir to the source directory. # STEP 2: Run the 'patch' program with this file as input. # #### End of Preamble #### #### Patch data follows #### diff -up 'build/Storable-2.16-0XWdnJ/t/croak.t' 'new.build/Storable-2.16-MSm6we/t/croak.t' Index: ./t/croak.t --- ./t/croak.t Fri Aug 16 21:15:56 2002 +++ ./t/croak.t Sun Jul 29 10:27:02 2007 @@ -10,10 +10,13 @@ sub BEGIN { chdir('t') if -d 't'; @INC = ('.', '../lib'); } - require Config; import Config; - if ($ENV{PERL_CORE} and $Config{'extensions'} !~ /\bStorable\b/) { - print "1..0 # Skip: Storable was not built\n"; - exit 0; + if ($ENV{PERL_CORE}) { + require Config; import Config; + %Config=%Config if 0; # cease -w + if ($Config{'extensions'} !~ /\bStorable\b/) { + print "1..0 # Skip: Storable was not built\n"; + exit 0; + } } } #### End of Patch data #### #### ApplyPatch data follows #### # Data version : 1.0 # Date generated : Sun Jul 29 11:07:01 2007 # Generated by : makepatch 2.03 # Recurse directories : Yes # Excluded files : (\A|/).*\~\Z # (\A|/).*\.a\Z # (\A|/).*\.bak\Z # (\A|/).*\.BAK\Z # (\A|/).*\.elc\Z # (\A|/).*\.exe\Z # (\A|/).*\.gz\Z # (\A|/).*\.ln\Z # (\A|/).*\.o\Z # (\A|/).*\.obj\Z # (\A|/).*\.olb\Z # (\A|/).*\.old\Z # (\A|/).*\.orig\Z # (\A|/).*\.rej\Z # (\A|/).*\.so\Z # (\A|/).*\.Z\Z # (\A|/)\.del\-.*\Z # (\A|/)\.make\.state\Z # (\A|/)\.nse_depinfo\Z # (\A|/)core\Z # (\A|/)tags\Z # (\A|/)TAGS\Z # p 't/croak.t' 997 1185697622 0100555 #### End of ApplyPatch data #### #### End of Patch kit [created: Sun Jul 29 11:07:01 2007] #### #### Patch checksum: 56 1936 53231 #### #### Checksum: 74 2560 39223 ####
Subject: Re: [rt.cpan.org #28537] PATCH for croak.t test failure
Date: Sun, 29 Jul 2007 19:15:57 +0530
To: Slaven_Rezic via RT <bug-Storable [...] rt.cpan.org>
From: Abhijit Menon-Sen <ams [...] toroid.org>
At 2007-07-29 05:10:46 -0400, bug-Storable@rt.cpan.org wrote: Show quoted text
> > The attached patch solves a possible failure in the croak.t test. It > seems that in some version Config.pm pre-loads Carp.pm, causing the > test to fail. The patch just makes sure that Config.pm is only loaded > when building Storable within core perl.
Thanks, applied. (#31664) -- ams