Login
is_script() now tries to check if the filename ends in ".PL".
authorJeffrey Ryan Thalhammer <jeff@imaginative-software.com>
Wed, 9 Jul 2008 19:37:30 +0000 (19:37 +0000)
committerJeffrey Ryan Thalhammer <jeff@imaginative-software.com>
Wed, 9 Jul 2008 19:37:30 +0000 (19:37 +0000)
commitdeb582127ac5b84ec1e2529eb61daa7c79867a71
treed35e1f235f38ff1de76631bdd221f316a06c054e
parent2849151b9300aeeba7871c8f76f3f49bc2491839
is_script() now tries to check if the filename ends in ".PL".
This prevents it from misclassifying Makefile.PL and Build.PL
and other ".PL" scripts as modules.  But if the document is
coming from a pipe (rather than a file) then the filename
won't be available and is_script() will still misclassify
the file.  Another approach may be to look for the absence
of a "package" statement, but I'm not sure that is very
reliable.  I suspect that many newbie developers and those
with very old legacy systems may have lots of libraries that
don't have any package statements.
Changes
lib/Perl/Critic/Utils.pm
t/05_utils.t