On 09/03/2012 20:16, Logan Bell wrote:
I did go and fix the pod checker issues, however, it might be good to just disable pod checker unless the tests are being run by an author. Below is an example of what I mean in 801-pod_checker.t.BEGIN { *plan skip_all => "Author mode not enabled SKIPPING"* unless $ENV{AUTHOR_MODE}; if ( $]< 5.010 ) { plan( 'skip_all', "Old Pod::Checker is buggy" ); } else { plan('no_plan'); } }
+1 for this.
Would anyone have any objection with this? It seems fairly reasonable since it feels a bit heavy handed to run this check on every install. Also, how difficult would it be to get out a new release up to CPAN I'd be willing to volunteer to do this if I knew the correct process.
I don't think the test failures a such an urgent problem. People who use one of the very recent versions of Pod::Parser that report failures should be smart enough to ignore them.
Nick
