Skip Menu |

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

Report information
The Basics
Id: 59737
Status: resolved
Priority: 0/
Queue: Data-Show

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

Bug Information
Severity: Critical
Broken in: 0.001001
Fixed in: 0.001002



Subject: Patches for (1) Data::Dump not listed as a dependency (2) Data::Dump broke backward compatibility
(1) The patches for Build.PL and Makefile.PL add Data::Dump >= 1.17 as a dependency. (2) The patch for show.t updates it to work with new Data::Dump 1.17 behavior (e.g. instead of displaying '[5, 6, 7, 8, 9, 10]' it will display '[5 .. 10]'). Thanks so much for your work on Data::Show. I have been using it since you described it at YAPC::NA this year.
Subject: Makefile.PL.diff
--- Makefile.PL 2010-07-27 09:50:38.000000000 -0500 +++ NewMakefile.PL 2010-07-27 09:48:51.000000000 -0500 @@ -10,6 +10,7 @@ PL_FILES => {}, PREREQ_PM => { 'Test::More' => 0, + 'Data::Dump' => 1.17, 'version' => 0, }, dist => { COMPRESS => 'gzip -9f', SUFFIX => 'gz', },
Subject: Build.PL.diff
--- Build.PL 2010-07-27 09:50:21.000000000 -0500 +++ NewBuild.PL 2010-07-27 09:48:43.000000000 -0500 @@ -9,6 +9,7 @@ dist_version_from => 'lib/Data/Show.pm', requires => { 'Test::More' => 0, + 'Data::Dump' => 1.17, 'version' => 0, }, add_to_cleanup => [ 'Data-Show-*' ],
Subject: show.t.diff
--- show.t 2010-07-27 10:00:58.000000000 -0500 +++ new_show.t 2010-07-27 10:00:27.000000000 -0500 @@ -37,7 +37,7 @@ __DATA__ ======( %foo )=============================[ 'show.t', line 12 ]====== - { foo => 1, food => 2, fool => 3, foon => [5, 6, 7, 8, 9, 10], foop => 4 } + { foo => 1, food => 2, fool => 3, foon => [5 .. 10], foop => 4 } ======( $/ )===============================[ 'show.t', line 13 ]======
Subject: Re: [rt.cpan.org #59737] Patches for (1) Data::Dump not listed as a dependency (2) Data::Dump broke backward compatibility
Date: Wed, 28 Jul 2010 07:49:32 +1000
To: bug-Data-Show [...] rt.cpan.org
From: Damian Conway <damian [...] conway.org>
Thanks, Christopher. Patches applied! Damian