Re: [PATCH] t960[34]: mark cvsimport tests as requiring perl

2014-11-18 Thread Jeff King
On Tue, Nov 18, 2014 at 10:56:22AM -0800, Jonathan Nieder wrote: > Jeff King wrote: > > > It would probably make sense to have these scripts just > > skip_all if NO_PERL is set, but I opted to follow the pattern > > set by t9600, etc. If somebody feels like spending time refactoring the > > cvsim

Re: [PATCH] t960[34]: mark cvsimport tests as requiring perl

2014-11-18 Thread Jonathan Nieder
Jeff King wrote: > It would probably make sense to have these scripts just > skip_all if NO_PERL is set, but I opted to follow the pattern > set by t9600, etc. If somebody feels like spending time refactoring the > cvsimport test harness, be my guest. Wouldn't it be a matter of the following, plu