Skip Menu |

This queue is for tickets about the Module-CPANTS-Analyse CPAN distribution.

Report information
The Basics
Id: 35116
Status: resolved
Priority: 0/
Queue: Module-CPANTS-Analyse

People
Owner: Nobody in particular
Requestors: h.m.brand [...] xs4all.nl
Cc:
AdminCc:

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



Subject: New metric: proper_libs
Date: Wed, 16 Apr 2008 15:38:04 +0200
To: bug-module-cpants-analyse [...] rt.cpan.org
From: "H.Merijn Brand" <h.m.brand [...] xs4all.nl>
* proper_libs Move your *.pm files in a directory named 'lib'. The directory structure should look like 'lib/Your/Module.pm' for a module named 'Your::Module'. NOOOOOOOOOOOOOOOOOOOO!!!! I absolutely disagree with this metric! thumbs down! For distributions with just a single .pm there is absolutely no harm in keeping it in the top-level directory. I'd even argue that it is right for up to 5 .pm files I know that Module::Build forces that structure upon you, but that is (only) one of the reasons I hate Module::Build -- H.Merijn Brand Amsterdam Perl Mongers (http://amsterdam.pm.org/) using & porting perl 5.6.2, 5.8.x, 5.10.x on HP-UX 10.20, 11.00, 11.11, & 11.23, SuSE 10.1 & 10.2, AIX 5.2, and Cygwin. http://qa.perl.org http://mirrors.develooper.com/hpux/ http://www.test-smoke.org http://www.goldmark.org/jeff/stupid-disclaimers/
Thanks for the report. This metric is marked as "extra" now (and thus won't affect the game). On Wed Apr 16 22:38:39 2008, h.m.brand@xs4all.nl wrote: Show quoted text
> * proper_libs > Move your *.pm files in a directory named 'lib'. The directory structure > should look like 'lib/Your/Module.pm' for a module named 'Your::Module'. > > NOOOOOOOOOOOOOOOOOOOO!!!! > > I absolutely disagree with this metric! > > thumbs down! > > For distributions with just a single .pm there is absolutely no harm in > keeping it in the top-level directory. I'd even argue that it is right > for up to 5 .pm files > > I know that Module::Build forces that structure upon you, but that is > (only) one of the reasons I hate Module::Build >