Re: [Module-build-general] RFC: Build test test_dirs=(list)

[Ken W. <ke...@ma...>]

On Tuesday, August 12, 2003, at 01:25  PM, David Carmean wrote:
>
> I was suggesting a new option, not a change to the default behavior: An
> argument named "test_dirs".  But unless I've missed a patch since 0.19 
> was
> posted to CPAN, M::B::Base::test_dirs already *does* recursively find 
> ".t" files
> in t/ .

Ooh, you're right!  I remember that setting off a flag in my head when 
I wrote it a long time ago, but then forgot about it.  It shouldn't 
scan recursively, for the reason Schwern mentions - as well as for 
compatibility reasons with MakeMaker layouts.

In some cases it might be handy to scan recursively though.  We'll have 
to think about a set of options that gives people flexibility in a few 
different ways.

> It would just make vim recognize them as Perl files by default :o) .   
> Speaking of magic, where did the .t convention come from anway?

It probably first appeared in the core, for the regression test scripts.

I agree with Schwern that the amount of magic in this area of M::B (and 
in most other areas) needs to stay very limited.  This can already be 
overridden by the user/subclasser anyway.  Plus, Test::Harness output 
doesn't/didn't look nice with .pl files.  PLUS, lots of people use .pl 
files to hold functions they share among their various test scripts.

  -Ken