Re: Overriding LN_S

[Ralf Wildenhues]

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