Cross-posted to module-build-general, p5p and perl-qa. Please trim CCs
if your reply isn't relevant to all the lists. I'm not subscribed to
module-build-general.
On Wed, Jun 02, 2004 at 12:09:01AM -0500, Ken Williams wrote:
> I've just released a new beta of Module::Build to CPAN. This is a
> sanity-check release on my way to 0.26. Changes since 0.25:
> - Added the 'testcover' action, which runs a test suite using
> Devel::Cover. [Dave Rolsky]
Nice. I'd been planning for something like that when I was happy with
Devel::Cover. But I imagine there will likely always be something more
I want to do, and that Devel::Cover is probably in a fairly usable state
at the moment.
A few questions about the code though, which I have reproduced below:
] sub ACTION_testcover {
] my ($self) = @_;
]
] unless ($self->find_module_by_name('Devel::Cover', [EMAIL PROTECTED])) {
] warn("Cannot run testcover action unless Devel::Cover is installed.\n");
] return;
] }
]
] $self->add_to_cleanup('coverage', 'cover_db');
"cover_db" is the default coverage database, but what is the file
"coverage"? I don't think that that is a file produced by Devel::Cover.
] local $Test::Harness::switches =
] local $Test::Harness::Switches =
] local $ENV{HARNESS_PERL_SWITCHES} = "-MDevel::Cover";
I think it is probably most useful for a coverage run to start with a
clean database by default. This would be accomplished, I think, by
$self->do_system('cover -delete');
] $self->depends_on('test');
] $self->do_system('cover');
] }
And now to the name, "testcover". Up to now, I have been using the
target "cover" for this purpose, and I know a couple of other projects
which have been doing the same. I also know other people have been
using Devel::Cover, but I don't know whether they have created a target
for this, and if so, what they have called it.
I'm not particularly attached to any one name, but I would like to
ensure consistency, so now seems like the right time to pick a good
target and stick to it.
I'd be interested in any opinions on this.
And then, at some point, add a similar rule to MakeMaker?
--
Paul Johnson - [EMAIL PROTECTED]
http://www.pjcj.net
