libtool patch from gettext

[Ralf Wildenhues]

I looked at a couple of ltmain patches from gettext, thanks to Earnie
Boyd for pointing them out to me.

The first one below I'd like to apply: return error from cwrapper if
exec*() failed (all branches).

Bruno, I believe your original ChangeLog entry for another patch is
wrong: it said
| * config/ltmain.sh (install): Use "ln -s -f" instead of
| "rm -f && ln -s" to make a symlink for a shared library. Take
| care of Solaris /bin/ln.
| Reported by Nelson H. F. Beebe <address@hidden>.

I have searched whatever I could find, and all I could find was a
request to try `ln -sf' before `rm -f && ln -s' in order to minimize
the time that the symlink would not be available, and no issue connected
with Solaris ln.  Can you enlighten us on this, so we can mention it in
the Autoconf Portability section?

Otherwise, I don't see much sense in replacing a race condition with a
slightly smaller race condition.

The third patch I found was one where with `chmod 777 .libs'.  I'd like
to fix relinking output in $DESTDIR/$prefix instead.

Thanks,
Ralf

2003-11-27  Bruno Haible  <address@hidden>

        * config/ltmain.sh (cwrappersource): return 127 if exec failed.

Index: libltdl/config/ltmain.m4sh
===================================================================
RCS file: /cvsroot/libtool/libtool/libltdl/config/ltmain.m4sh,v
retrieving revision 1.2
diff -u -r1.2 ltmain.m4sh
--- libltdl/config/ltmain.m4sh  24 Aug 2005 04:51:41 -0000      1.2
+++ libltdl/config/ltmain.m4sh  24 Aug 2005 08:03:37 -0000
@@ -5787,7 +5787,7 @@
            esac
 
            cat >> $cwrappersource <<"EOF"
-  return 0;
+  return 127;
 }
 
 void *