Skip Menu |

This queue is for tickets about the Data-Dump-Streamer CPAN distribution.

Report information
The Basics
Id: 89674
Status: resolved
Priority: 0/
Queue: Data-Dump-Streamer

People
Owner: Nobody in particular
Requestors: chisel [...] chizography.net
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 2.36
Fixed in: 2.37



Subject: Tests rely on modules only listed as 'recommends' in Build.PL
I would like to see PadWalker being promoted to a build_requires or a requires; or skip the tests that require it present to pass: [5011][c.wright@xtdc1-chz:Data-Dump-Streamer-2.36][CPAN-updates⚡]➔ prove -l t/lexicals.t t/lexicals.t .. # # PadWalker isn't installed t/lexicals.t .. 1/14 # Error: # Data::Dump::Streamer->Data(@_)->Out() failed dump: # normalize ( scalar $obj->Data(@_)->Out() ) # Can't locate object method "ARRAY" via package "B::NULL" at /home/c.wright/development/dave/Data-Dump-Streamer-2.36/lib/Data/Dump/Streamer.pm line 3701. # Failed test 'Unimplemented code' # at t/test_helper.pl line 527. # Looks like you failed 1 test of 14. t/lexicals.t .. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/14 subtests Test Summary Report ------------------- t/lexicals.t (Wstat: 256 Tests: 14 Failed: 1) Failed test: 7 Non-zero exit status: 1 Files=1, Tests=14, 0 wallclock secs ( 0.02 usr 0.00 sys + 0.09 cusr 0.01 csys = 0.12 CPU) Result: FAIL [5012][c.wright@xtdc1-chz:Data-Dump-Streamer-2.36][CPAN-updates⚡]➔ cat lib/PadWalker.pm package PadWalker; die 'bibble'; 1; [5013][c.wright@xtdc1-chz:Data-Dump-Streamer-2.36][CPAN-updates⚡]➔ mv lib/PadWalker.pm{,.bak} [5014][c.wright@xtdc1-chz:Data-Dump-Streamer-2.36][CPAN-updates⚡]➔ prove -l t/lexicals.t t/lexicals.t .. # # PadWalker 1.96 is installed t/lexicals.t .. ok All tests successful. Files=1, Tests=14, 0 wallclock secs ( 0.02 usr 0.00 sys + 0.09 cusr 0.00 csys = 0.11 CPU) Result: PASS [5015][c.wright@xtdc1-chz:Data-Dump-Streamer-2.36][CPAN-updates⚡]➔
Subject: Data-Dump-Streamer-2.36-PadWalker.patch
diff -Naur Data-Dump-Streamer-2.36-orig/Build.PL Data-Dump-Streamer-2.36/Build.PL --- Data-Dump-Streamer-2.36-orig/Build.PL 2013-10-22 09:51:13.000000000 +0100 +++ Data-Dump-Streamer-2.36/Build.PL 2013-10-22 09:52:19.000000000 +0100 @@ -27,6 +27,7 @@ 'Data::Dumper' => 0, 'Devel::Peek' => 0, 'ExtUtils::CBuilder' => 0, + 'PadWalker' => '0.99', 'Symbol' => 0, 'Test::More' => 0, 'Text::Abbrev' => 0, @@ -75,7 +76,6 @@ # CPAN 'Algorithm::Diff' => 0, - 'PadWalker' => '0.99', # optional for testing 'JSON::XS' => 0,
On 2013-10-22 01:57:02, CHISEL wrote: Show quoted text
> I would like to see PadWalker being promoted to a build_requires or a > requires; or skip the tests that require it present to pass:
use Test::Requires 'PadWalker';
On 2013-10-22 04:57:02, CHISEL wrote: Show quoted text
> I would like to see PadWalker being promoted to a build_requires or a > requires; or skip the tests that require it present to pass:
PadWalker is still not required if the patch in https://rt.cpan.org/Ticket/Display.html?id=79127 was applied. Regards, Slaven
On 2013-10-22 04:57:02, CHISEL wrote: Show quoted text
> I would like to see PadWalker being promoted to a build_requires or a > requires; or skip the tests that require it present to pass:
[...] Problem should be fixed in the latest 2.37 release. Regards, Slaven