Login
No longer generating the PolicySummary when running the "test" action.
authorJeffrey Ryan Thalhammer <jeff@imaginative-software.com>
Sun, 24 Jan 2010 04:56:40 +0000 (04:56 +0000)
committerJeffrey Ryan Thalhammer <jeff@imaginative-software.com>
Sun, 24 Jan 2010 04:56:40 +0000 (04:56 +0000)
This was causing all the build failures in the last release, because
the PolicySummary had already been generated and had read-only
permissions when a user receives the distro.  So running "./Build test"
would fail because it would try to regenerate the read-only file.

I also noticed that we're deleting and regenerating the MANIFEST
every time that you "./Build test".  This seems like a convenience
for us authors, but for end-users, it seems like awfully weird behavior.
So I took that out as well.

So now, the PolicySummary and MANIFEST are generated only when running
authortest, which always happens before making a distro.  If an author
wants to run some other tests that require the MANIFEST, then it is up
to them to run the "manifest" target.

At least, this is what makes sense to me.  I know Elliot has strong
opionions about this, and went to great lengths to configure this
build arrangement.  So he may think differently.

inc/Perl/Critic/Module/Build.pm

index 82b87b3..91ef0d7 100644 (file)
@@ -21,16 +21,6 @@ use Perl::Critic::PolicySummaryGenerator qw< generate_policy_summary >;
 use base 'Module::Build';
 
 
-sub ACTION_test {
-    my ($self, @arguments) = @_;
-
-    $self->depends_on('policysummary');
-    $self->depends_on('manifest');
-
-    return $self->SUPER::ACTION_test(@arguments);
-}
-
-
 sub ACTION_authortest {
     my ($self) = @_;