[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

libtool patch from gettext

From: Ralf Wildenhues
Subject: libtool patch from gettext
Date: Wed, 24 Aug 2005 13:26:15 +0200
User-agent: Mutt/1.4.1i

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/ (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.


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

        * config/ (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 @@
            cat >> $cwrappersource <<"EOF"
-  return 0;
+  return 127;
 void *


reply via email to

[Prev in Thread] Current Thread [Next in Thread]