From: Dave S. <D.T...@cs...> - 2005-09-20 08:47:13
|
On Sun, 2005-09-18 at 21:11 +0200, Magnus Fromreide wrote: > On Fri, Sep 16, 2005 at 04:06:06PM +0100, Dave Shield wrote: > > Is there anything in particular > > that you think "messes up configure.in" unnecessarily? > > It is mainly that it changes the style used.... > but the patch breaks this and doesn't kill of the comments > that suggest that ordering. It'd be nice to To Things Right, I agree. But that doesn't feel a particularly major problem. The configure script is in such a mess already, a little bit extra won't hurt! I'd much prefer to see this patched now (and fixed properly later), rather than go through another release cycle with sort-of-broken code. We currently get this problem reported fairly regularly, and it would be useful to stem that. (Plus if Thomas is correct, then it'll become vital to fix it at some point fairly soon, so we might as well start now). I propose that this patch be applied ASAP, and we see what happens. Dave |