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)
commit00b7b1ab628b2f743370380ed11a213ccf058c97
tree147ccd7475071acc42ac90cefbe421c3dba5af34
parentd1c572805c7818e496e13a29d2ec14f4635ade6f
No longer generating the PolicySummary when running the "test" action.
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