* 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