[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/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 *
- libtool patch from gettext,
Ralf Wildenhues <=