Skip Menu |
 

This queue is for tickets about the libwww-perl CPAN distribution.

Report information
The Basics
Id: 94948
Status: resolved
Priority: 0/
Queue: libwww-perl

People
Owner: Nobody in particular
Requestors: ppisar [...] redhat.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 6.06
Fixed in: (no value)



Subject: Useless run-time dependency on Data::Dump
Download (untitled) / with headers
text/plain 237b
6.06 META.json lists Data::Dump in prereqs{runtime}{requires}. However the module is not used anywhere. Even Changes states "Removed stray Data::Dump reference from test suite." I recommend to remove the module from the dependency list.
Download (untitled) / with headers
text/plain 416b
On Wed Apr 23 01:12:15 2014, ppisar wrote: Show quoted text
> 6.06 META.json lists Data::Dump in prereqs{runtime}{requires}. However > the module is not used anywhere.
Actually, I've added it to the dependency list because the test suite is using it: https://github.com/libwww-perl/libwww-perl/blob/master/t/local/httpsub.t Arguably, this could be done without Data::Dump and Data::Dump be removed from the prereq list again.
Subject: Re: [rt.cpan.org #94948] Useless run-time dependency on Data::Dump
Date: Wed, 23 Apr 2014 09:46:13 +0200
To: Michael_Schilli via RT <bug-libwww-perl [...] rt.cpan.org>
From: Petr Pisar <ppisar [...] redhat.com>
Download (untitled) / with headers
text/plain 519b
On Wed, Apr 23, 2014 at 01:35:19AM -0400, Michael_Schilli via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=94948 > > > On Wed Apr 23 01:12:15 2014, ppisar wrote:
> > 6.06 META.json lists Data::Dump in prereqs{runtime}{requires}. However > > the module is not used anywhere.
> > Actually, I've added it to the dependency list because the test suite is > using it: > > https://github.com/libwww-perl/libwww-perl/blob/master/t/local/httpsub.t >
The file is missing from the 6.06 release. -- Petr
Download (untitled)
application/pgp-signature 230b

Message body not shown because it is not plain text.

Download (untitled) / with headers
text/plain 272b
On Wed Apr 23 03:46:30 2014, ppisar wrote: Show quoted text
> The file is missing from the 6.06 release.
Good catch, while it's checked into the repo, the MANIFEST file doesn't have it and therefore it's missing from the dist tarball. One more reason to remove the dependency, thanks!


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.