On July 15, 2010, Andy Lester wrote:
> >>    prove --ext=.phpt --ext=.t --exec=/usr/local/bin/my-test-dispatcher
> > 
> > Isn't that just
> > 
> > --- a/lib/App/Prove.pm
> > +++ b/lib/App/Prove.pm
> > @@ -214,7 +214,7 @@ sub process_args {
> > 
> >             'count!'       => \$self->{show_count},
> >             'c'            => \$self->{color},
> >             'D|dry'        => \$self->{dry},
> > 
> > -            'ext=s'        => \$self->{extension},
> > +            'ext=s+'       => \$self->{extension},
> > 
> >             'harness=s'    => \$self->{harness},
> >             'ignore-exit'  => \$self->{ignore_exit},
> >             'source=s@'    => $self->{sources},
> 
> Might be that simple, but I don't know the guts of prove any more.  That's
> why I asked.  I didn't want to make this a CYJ.

Unfortunately, I don't think its quite that simple (seemed too good to be true 
so I had to take a quick check and try it out).

Although setting it to "ext=s@" lets you hand in multiple extensions on the 
cmd line, they're not used properly when it gets handed down the line to 
App::Prove::State.

That said, though, it doesn't look _that_ hard to make this work, just not 
quite as simple as above.

-- 
Graham TerMarsch
Howling Frog Internet Development, Inc.

Reply via email to