* Stefano Lattarini wrote on Mon, Sep 06, 2010 at 01:05:25AM CEST:
> On Sunday 05 September 2010, Ralf Wildenhues wrote:
> > * Stefano Lattarini wrote on Fri, Sep 03, 2010 at 12:14:18AM CEST:
> > > 
> > > -srcdir=${srcdir-'@abs_srcdir@'}
> > > +srcdir='@abs_srcdir@'
> > 
> > Can you explain why this is not needed conditionally any more?
> For consistency with the variable `$top_scrdir' introduced
> in the PATCH 2/4, which is unconditonally defined.  Also, while in my 
> endless tweaking an experimentation with the testsuite I've wanted to 
> and/or resorted to override many things, I not once needed or wished 
> to override $(srcdir).  And the uncondional definition let us remove
> # the code below for srcdir "absolutization".
> > I mean, was there a reason we didn't already do that last time?
> Which last time?

Well, in v1.11-139-g6fee87c and before that in v1.11-138-gc6bbc45
the above line was move resp. introduced.  Was that a buglet in the
latter patch?

If yes, then this patch is fine.  The other updated patches 2 and 3 in
this series are fine anyway.

Thanks,
Ralf

Reply via email to