[Bug 461350] cpanspec nearly always misses the BuildRequires: perl(Test::More)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=461350





--- Comment #4 from Steven Pritchard <steve@xxxxxxxxx>  2009-01-21 20:53:21 EDT ---
(In reply to comment #3)
> THe CPANTS test suite makes use of Module::ExtractUse which seems to do a
> pretty good job (at least on the modules I maintain it appears accurate). The
> CPANTS results also suggest that on the order of 1/3 of all modules have
> missing PreReqs declared in their META.yml/Makefile.PL vs their actual use. So,
> IMHO it'd be worth making use of module like this to extract usage. 

How about just extracting module use from the tests?  The current version in
CVS (
http://cpanspec.cvs.sourceforge.net/viewvc/*checkout*/cpanspec/cpanspec/cpanspec
) has a simple stab at this idea.

Interestingly, see how badly it blows up on Module::ExtractUse's tests.  :-)

> Perhaps if this detected modules not declared in the META.yml, it could add the
> BR to the RPM spec, along with a comment informing the packager that they
> should double check this module usage manually.

That would take more code, but it would probably be better in the long run...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
Fedora-perl-devel-list mailing list
Fedora-perl-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-perl-devel-list

[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Legacy Announce]     [Fedora PHP Devel]     [Kernel Devel]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Big List of Linux Books]     [Gimp]     [Yosemite Information]
  Powered by Linux