Re: [patch] make test runs test on included modules

2010-06-23 Thread Fred Moyer
I just committed this patch r957309. I opted for this version rather than the perl generated targets in this thread only because this one fits in snugly with the existing style. I poked this several different ways to make sure it is doing everything from running the tests to installing the module

Re: [patch] make test runs test on included modules

2010-06-15 Thread Philippe M. Chiasson
On 10-06-15 15:14 , Adam Prime wrote: > Philippe M. Chiasson wrote: >> How about this cute trick using make -k ? Quick testing seems to >> indicate it ends up doing the right thing for me. >> > > seems reasonable to me, is there any make magic that can be done to > automatically create all the run

Re: [patch] make test runs test on included modules

2010-06-15 Thread Adam Prime
Philippe M. Chiasson wrote: How about this cute trick using make -k ? Quick testing seems to indicate it ends up doing the right thing for me. seems reasonable to me, is there any make magic that can be done to automatically create all the run_subtests targets? Adam ---

Re: [patch] make test runs test on included modules

2010-06-15 Thread Philippe M. Chiasson
On 10-06-14 22:47 , Philip M. Gollucci wrote: > On 06/14/10 19:46, Philippe M. Chiasson wrote: >> On 10-06-14 15:16 , Fred Moyer wrote: >>> I put this patch together as part of the todo list for 2.0.5. Gozer, >>> is this what we are looking for? >>> >>> Index: Makefile.PL >>> =

Re: [patch] make test runs test on included modules

2010-06-14 Thread Philip M. Gollucci
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/14/10 19:46, Philippe M. Chiasson wrote: > On 10-06-14 15:16 , Fred Moyer wrote: >> I put this patch together as part of the todo list for 2.0.5. Gozer, >> is this what we are looking for? >> >> Index: Makefile.PL >>

Re: [patch] make test runs test on included modules

2010-06-14 Thread Philippe M. Chiasson
On 10-06-14 15:16 , Fred Moyer wrote: > I put this patch together as part of the todo list for 2.0.5. Gozer, > is this what we are looking for? > > Index: Makefile.PL > === > --- Makefile.PL (revision 953555) > +++ Makefile.PL (worki

[patch] make test runs test on included modules

2010-06-14 Thread Fred Moyer
I put this patch together as part of the todo list for 2.0.5. Gozer, is this what we are looking for? Index: Makefile.PL === --- Makefile.PL (revision 953555) +++ Makefile.PL (working copy) @@ -786,6 +786,8 @@ $(FULLPERL) -I$