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

Attachments


Subject: Tests rely on modules only listed as 'recommends' in Build.PL
Download (untitled) / with headers
text/plain 1.5k
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,
Download (untitled) / with headers
text/plain 205b
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';
Download (untitled) / with headers
text/plain 306b
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
Download (untitled) / with headers
text/plain 254b
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


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.