Hi Peter, * Peter Rosin wrote on Fri, Oct 15, 2010 at 09:28:53AM CEST: > I haven't actually tested the patch, but it seems trivial enough.
Bzzt. You are just barely saved yourself by admitting it upfront! ;-> > --- a/lib/m4sugar/m4sh.m4 > +++ b/lib/m4sugar/m4sh.m4 > else > - as_ln_s='cp -p' > -fi > -rm -f conf$$ conf$$.exe conf$$.dir/conf$$.file conf$$.file > -rmdir conf$$.dir 2>/dev/null > + as_ln_s=$LN_S > +endif Shell if ends with fi, not endif. > ])# _AS_LN_S_PREPARE With that fixed, I ran the testsuite on GNU/Linux once without LN_S set, once with export LN_S='cp -p', both successfully, but with LN_S=bla, the only failure is from the foreign.at:Libtool test, because libtoolize honors $LN_S. I think a testsuite addition that ensures that LN_S is honored for both AC_PROG_LN_S and for AS_LN_S is in order; we have already regressed on this once, a few years ago. Please ping me if you want me to write one. Thanks, Ralf