Stories
Slash Boxes
Comments
NOTE: use Perl; is on undef hiatus. You can read content, but you can't post it. More info will be forthcoming forthcomingly.

All the Perl that's Practical to Extract and Report

use Perl Log In

Log In

[ Create a new account ]

domm (4030)

domm
  (email not shown publicly)
http://domm.plix.at/

Just in case you like to know, I'm currently full-time father of 2 kids, half-time Perl hacker, sort-of DJ, bicyclist, no longer dreadlocked and 33 years old

I'm also head of Vienna.pm [pm.org], maintainer of the CPANTS [perl.org] project, member of the TPF Grants Commitee [perlfoundation.org] and the YAPC Europe Foundation [yapceurope.org].

I've got stuff on CPAN [cpan.org], held various talks [domm.plix.at] and organised YAPC::Europe 2007 in Vienna [yapceurope.org].

Journal of domm (4030)

Tuesday October 30, 2007
04:16 PM

CPANTS news

[ #34792 ]

(Crossposted from http://cpants.perl.org/news.html)

The view for a dist is now broken up into several subpages. This makes the data more easy to read. I've also fixed some minor problems in the list of prereqs etc (links are now going to the dist each module is in).

I've added a new metric: use_warnings. This metric checks if all of your modules include a line like 'use warnings;'. As this does not work for old Perls, I've made it an optional metric (for now..).

A patch submitted by Adriano Ferreira enhances MANIFEST-checking (and removes a bug caused by chdir). See RT #28982 for details.

Andy Armstrong suggested to also look in xt/ for tests, because this is becoming the new place for optional author tests (like pod-coverage). Which I implemented...

The META.yml checks submitted by Barbie turn up a lot of interesting results:

  • Module::Install creates META.yml files that do not conform to the spec, because the author field has to be a list (but Module::Install creates it as a string)
  • There seems to be a problem with version numbers with a 'v' (as in 'v1.42') in prereq (but this might be a problem of the testing code?)
  • Currently, 10550 of 13145 dists do not come with a valid META.yml. That means that only 20% of the META.yml files conform to the spec (or CPANTS is too strict...)

I've also re-analysed all of CPAN. The only thing left to do now (besides adding more metrics) is to make the various graphs work again...

The Fine Print: The following comments are owned by whoever posted them. We are not responsible for them in any way.
 Full
 Abbreviated
 Hidden
More | Login | Reply
Loading... please wait.
  • After all, it's things like this that CPANTS was created for in the first place.
    • SO, does this mean you'll fix Module::Install?

      My naive attempts at fixing it myself broke it even more >=(

    • I'd second this view.

      If you aren't going to have a strict conformance to the spec, then there is little point in having the checks at all.

      With more requests to add metadata to the spec, if you want people to have confidence in the META.yml bundled with distributions, then authors need to be made aware of the areas that they don't conform. CPANTS is the ideal place for that.

  • Here's what I'd recommend doing.

    Firstly, you need a new "has_perl_version" test.

    This would check that the distribution explicitly declares (presumably in it's META.yml) the version of perl that the distribution depends on.

    Secondly, once you have that in place, make the use warnings logic something like...

    if ( dist needs 5.6+ ) {
        if ( has_warnings ) {
            return pass;
        } else {
            return fail;
        }
    } else {
        return

  • I think you may want to revise the code for checking use_warning. CPANTS fails Finance::Currency::Convert::XE [perl.org], but 'use warnings' is right there [cpan.org] on line 5 and has been since it existed! It's the only module in the distribution, so it isn't referencing something else. However, this also raised a thought about whether you check modules under the /t (or /xt) directories. Some of those modules may not have 'use warnings' for a good reason in order to test aspects of the distribution.
    • The code is ok, as it only checks .pm files in either the basedir, or in lib. If there are no modules in these places, CPANTS locates all .pm files, but skips x?t, test and inc paths.

      In this case the problem is that CPANTS didn't pick up the correct CPANTS yaml file. Instead of using the correct one (of 0.14, btw), it used an outdated yaml file left over from a CPANTS run where the 'use_warnings' metric wasn't even implemented.

      So, yes, there is a bug in CPANTS, but not in 'use_warnings'. I'll try to fix it
    • Fixed in the current version of cpants. New data is online, also.

      This distribution passes all CPANTS tests. Yay! [perl.org]

      (using use.perl as a bugtracker)++

  • Moose enables warnings and strict automagically (they're turned on via Moose's import), Test::Kwalitee already dislikes this so I have to (redundantly) add use strict; to everything ... are you saying I'm gonna get my CPANTS score docked now too cause I don't add a redundant use warnings; as well?
    • Who cares about your CPANTS score though? I look at it to see if I stupidly forgot something, not to see if I have a good score.

      "use warnings" is a dumb metric because it has a runtime performance hit. If I use it during development (or make test), that's one thing... but imposing it on users of the CPAN module is not necessarily a good thing.

      I do it anyway, out of laziness, but it's not necessarily a good idea.