David Golden wrote: > On 11/1/06, Michael G Schwern <[EMAIL PROTECTED]> wrote: >> Contact all the authors making use of it and suggest they switch to .t >> files. Maybe I could throw a warning into MakeMaker. Also a kwalitee >> check sounds like a good idea "not_only_test.pl" or something. > > It shouldn't be "not_only". If there is t/*.t and test.pl, > ExtUtils::MakeMaker runs them separately, first passing t/*.t to > Test::Harness and then separately running test.pl. If the > Test::Harness run passes, then we're right back to the same problem as > if test.pl was just by itself.
As pointed out earlier, some distributions use test.pl as a general "do some interesting stuff" program which are not necessarily tests. DBI, for example, uses it to do benchmarking. I don't believe this usage to be widespread at all. That can be taken as both an argument for making it a kwalitee test, "Its almost always a mistake to have both t/*.t and test.pl" or against "So few people actually do this, let's leave them be". I'm doing a CPAN scan now to see which distributions only have a test.pl and which have both.