Re: [PATCH] test defs: add subroutine for input unindenting

2011-02-15 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Wed, Feb 16, 2011 at 12:29:02AM CET: > On Tuesday 15 February 2011, Ralf Wildenhues wrote: > > The space after : is not portable. The sed script could be shortened to > > sed -n "/[^ $tab].*$/{ > > s///p > > d > > }" > > > Did you me

Re: [PATCH] test defs: add subroutine for input unindenting

2011-02-15 Thread Stefano Lattarini
Hello Ralf. On Tuesday 15 February 2011, Ralf Wildenhues wrote: > * Stefano Lattarini wrote on Tue, Feb 15, 2011 at 01:36:16PM CET: > > OK, here's my shot at it. The implementation might be suboptimal, > > but since it wasn't completely obvious to get right, I'd rather not > > tweak it anymore, u

Re: [PATCH] test defs: add subroutine for input unindenting

2011-02-15 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Tue, Feb 15, 2011 at 01:36:16PM CET: > OK, here's my shot at it. The implementation might be suboptimal, > but since it wasn't completely obvious to get right, I'd rather not > tweak it anymore, until there's a real need at least. Lemme tweak it for you. ;-) > (By t

Re: [Ping PATCHES] {master} Optimize tests `instspc-*.test' for speed.

2011-02-15 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Tue, Feb 15, 2011 at 10:21:36AM CET: > On Monday 14 February 2011, Ralf Wildenhues wrote: > > > --- /dev/null > > > +++ b/tests/instspc-data.test > > > > > +# Helper testcase which generate input data for the other test > > > +# `instspc-*.test'. It basically delegate

[PATCH] test defs: add subroutine for input unindenting (was: Re: [Ping PATCHES] {master} Optimize tests `instspc-*.test' for speed.)

2011-02-15 Thread Stefano Lattarini
On Tuesday 15 February 2011, Stefano Lattarini wrote: > On Monday 14 February 2011, Ralf Wildenhues wrote: > > Hi Stefano, > > > > a while ago ... > > > > * Stefano Lattarini wrote on Tue, Jan 25, 2011 at 06:38:50PM CET: > > > > > >

Re: [Ping PATCHES] {master} Optimize tests `instspc-*.test' for speed.

2011-02-15 Thread Stefano Lattarini
On Monday 14 February 2011, Ralf Wildenhues wrote: > Hi Stefano, > > a while ago ... > > * Stefano Lattarini wrote on Tue, Jan 25, 2011 at 06:38:50PM CET: > > > >

Re: bug#7944: Should AM_PATH_PYTHON call AC_ARG_VAR?

2011-02-15 Thread Stefano Lattarini
On Tuesday 15 February 2011, Ralf Wildenhues wrote: > > Sorry, I'm probably just making myself not clear. Here's a patch that I > would squash in on top of your patch, and that I think addresses all > issues that I mention in my review. It basically reverses the doc part > of your patch and then