Skip Menu |
 
rt.cpan.org will be shut down on March 1st, 2021.

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Test-TCP CPAN distribution.

Report information
The Basics
Id: 60362
Status: resolved
Priority: 0/
Queue: Test-TCP

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

Bug Information
Severity: Unimportant
Broken in: 1.00
Fixed in: (no value)



Subject: Possibly needless Deps/Files.
Download (untitled) / with headers
text/plain 634b
I was looking through the code making sure our (Gentoo's ) package management deps properly corresponded to your dists deps, and found a few inconsistencies that might be helpful to remedy. 1. There's fully fledged copies of Test::Base and YAML in inc/ , however, these appear to be used nowhere in the code. 2. As a result of the above, there is a DEP declared in META.yml on Filter::Util::Call , which I can't see a need for. 3. As a result of the above, test_requires 'YAML' in Makefile.PL may also not be needed. You don't have to remedy this, but it would make for a smaller and cleaner distribution if you did =). Thanks.
Subject: Re: [rt.cpan.org #60362] Possibly needless Deps/Files.
Date: Sun, 15 Aug 2010 22:29:07 +0900
To: bug-Test-TCP [...] rt.cpan.org
From: Tokuhiro Matsuno <tokuhirom [...] gmail.com>
Download (untitled) / with headers
text/plain 1.3k
Hi ketnl, Filter::Util::Call was first released with perl v5.7.3. And other modules are only appeared in inc/. It is not issue. Package maintainer should ignore inc/ and test_requires. But so, I removed deps for these modules for clean up. regards, On Sun, Aug 15, 2010 at 7:52 PM, Kent Fredric via RT <bug-Test-TCP@rt.cpan.org> wrote: Show quoted text
> Sun Aug 15 06:52:07 2010: Request 60362 was acted upon. > Transaction: Ticket created by KENTNL >       Queue: Test-TCP >     Subject: Possibly needless  Deps/Files. >   Broken in: 1.00 >    Severity: Unimportant >       Owner: Nobody >  Requestors: KENTNL@cpan.org >      Status: new >  Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=60362 > > > > I was looking through the code making sure our (Gentoo's ) package > management deps properly corresponded to your dists deps, and found a > few inconsistencies that might be helpful to remedy. > > 1. There's fully fledged copies of Test::Base and YAML  in inc/ , > however, these appear to be used nowhere in the code. > > 2. As a result of the above, there is a DEP declared in META.yml on > Filter::Util::Call  , which I can't see a need for. > > 3. As a result of the above, test_requires 'YAML' in Makefile.PL may > also not be needed. > > You don't have to remedy this, but it would make for a smaller and > cleaner distribution if you did =). > > Thanks. >
fixed in 1.01
Download (untitled) / with headers
text/plain 438b
On Sun Aug 15 09:29:25 2010, tokuhirom@gmail.com wrote: Show quoted text
> Filter::Util::Call was first released with perl v5.7.3. And other > modules are only appeared in inc/. It is not issue. > Package maintainer should ignore inc/ and test_requires
We're a source-based distribution, and we like to let our users run tests during installation if they want to =) Show quoted text
> > But so, I removed deps for these modules for clean up.
All the same, thanks =).
Silly RT. I didn't tell it to re-open, but it reopened anyway !


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.