[perl #40598] [PATCH] Make perlcritic.t barf more nicely when Test::Perl::Critic not installed

2006-10-26 Thread Paul Cochrane
# New Ticket Created by Paul Cochrane # Please include the string: [perl #40598] # in the subject line of all future correspondence about this issue. # URL: http://rt.perl.org/rt3/Ticket/Display.html?id=40598 Hi, this patch ensures that when t/codingstd/perlcritic.t can't find

Re: [perl #40598] [PATCH] Make perlcritic.t barf more nicely when Test::Perl::Critic not installed

2006-10-26 Thread Chris Dolan
On Oct 26, 2006, at 5:21 AM, Paul Cochrane (via RT) wrote: this patch ensures that when t/codingstd/perlcritic.t can't find Test::Perl::Critic, that instead of exiting with a Can't locate Test/Perl/Critic.pm in @INC compile-time error, it reports that it is skipping the test with the message

Re: [perl #40598] [PATCH] Make perlcritic.t barf more nicely when Test::Perl::Critic not installed

2006-10-26 Thread Paul Cochrane
A better implementation of that patch would be: BEGIN { eval { require Test::Perl::Critic; }; if ($@) { plan skip_all = 'Test::Perl::Critic not installed'; } Test::Perl::Critic-import(-verbose = 7); Thanks Chris! I didn't know how to maintain the verbose