Re: Libtool HEAD on Windows.

[Ralf Wildenhues]

* Duft Markus wrote on Mon, Mar 03, 2008 at 09:33:44AM CET:
> >> 
> >> Attached is both a log for configure/build and the complete
> >> testsuite, and a patch against current HEAD.
> > 
> > Has anybody looked into this yet? I'm afraid this won't apply anymore
> > when it's finally reviewed.
> 
> Just a comment would have been cool, and if it's something like "we
> don't have time right now". However this patch is fairly easy to review
> i think, since nearly every change is inside a case for winnt...
> 
> Still waiting for _something_...

Sorry about being completely silent.  Your patch is intrusive in that it
changes more than should be necessary, it changes code that should not
be changed.  The ideal is: only libtool.m4 and ltdl.m4 contain system
specifics, all other code is generic.  Now I know that we're far from
this ideal, but every step further away is one that should be
scrutinized.

Also, some of the testsuite changes look like they are papering over the
failure rather than fix the issue.  The template.at changes fall into
this category.

Last but not least, I don't know if you noticed that Libtool was in
regression fixes mode up to the 2.2 release (yeah, I know it was an ugly
long time).  Given limited time, I for one simply ignored everything
that wasn't fixing a regression or easy to verify as safe.

I might give a detailed review on your patch, but that may be some days.

Cheers,
Ralf