Skip Menu |

This queue is for tickets about the Devel-Comments CPAN distribution.

Report information
The Basics
Id: 73196
Status: rejected
Priority: 0/
Queue: Devel-Comments

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

Bug Information
Severity: (no value)
Broken in: v1.1.4
Fixed in: (no value)



Subject: META.yml recommend its own modules
META.yml says: provides: [...] IO::Capture::Sayfix: file: lib/IO/Capture/Sayfix.pm version: 0.05 IO::Capture::Tellfix: file: lib/IO/Capture/Tellfix.pm version: 0.05 recommends: IO::Capture::Sayfix: 0 [...] IO::Capture::Tellfix: 0 The recommends section should not list modules that are provided only and only by the just-building package.
ppisar, You are probably right but this will not be fixed directly. I'll explain. First, 'recommends' isn't quite right anyway. The modules in question should be 'build_recommends'; they are only required to run some tests. But unfortunately Module::Build doesn't support this (yet). Second, IO::Capture::*fix was a terrible, nasty, temporary kludge I threw in, without much forethought, hoping the primary author would pick up on the issues I raised; fix and release. I don't think this has, or will, happen; and I shouldn't impose this particular nasty kludge for any length of time. I should have disposed of this long ago. Third, I'm now opposed to IO::Capture* entirely. Test::Trap is my current preference for pressure-cooker testing and will be used in the 1.2.0 release of Devel::Comments. Since this works as is, I won't be distributing it sneakily in D::C. I will probably still list Test::Trap as a 'recommends' even though it will only be used in testing, since I feel that to 'build_requires' a new dep would demand I bump major version. Thank you for your interest, XIONG