Re: Tru64/OSF: BIN_SH revisited

[Paul Eggert]

Stepan Kasal [EMAIL PROTECTED] writes:

 These two changes would make the following patch; OK to commit?

Yes, except you need to modify the Autoconf documentation accordingly.

HEAD fails mdemo-exec on darwin6 (Mac OS X 10.2)

[Peter O'Gorman]

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi,
For some reason I switched on my ancient powermac g3 and ran the libtool
testsuite on it. mdemo-exec fails, well, lt_dlopen(0,...) uses the wrong
loader, it uses the preopen loader where it should be using the dyld loader.

Changing LT_DLLOADER_APPEND to LT_DLLOADER_PREPEND in dyld.c fixes it, but
I'm not sure that it is the appropriate fix (which is why I'm asking for
approval rather than applying as obvious).

Peter
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.0 (Darwin)

iQCVAwUBRBylpriDAg3OZTLPAQK8AQP+PL3X5nbdTXTp9S5eFz1evCthg8uv2M/V
9ysZAb9ttREK2V5T0uHzg0xTxBXKvi3zQCP84kYKAGU9oOCbK1YyX5X5/9MLKjAn
AejEoUbDcUYJ9SRBiuokBBiflrTCb5+N5Tj5M7lWXTCgUE8rQcMC3AwW8PE/XD45
bL58z3hWNUQ=
=20y1
-END PGP SIGNATURE-
Index: ChangeLog
2006-03-19  Peter O'Gorman  [EMAIL PROTECTED]

* libltdl/loaders/dyld.c (get_vtable) [darwin]: Use
LT_DLLOADER_PREPEND.

from  Ralf Wildenhues  [EMAIL PROTECTED]
Index: libltdl/loaders/dyld.c
===
RCS file: /cvsroot/libtool/libtool/libltdl/loaders/dyld.c,v
retrieving revision 1.5
diff -u -3 -p -u -r1.5 dyld.c
--- libltdl/loaders/dyld.c 22 Apr 2005 10:10:30 - 1.5
+++ libltdl/loaders/dyld.c 18 Mar 2006 23:57:12 -
@@ -70,7 +70,7 @@ get_vtable (lt_user_data loader_data)
   vtable-module_close = vm_close;
   vtable-find_sym = vm_sym;
   vtable-dlloader_data= loader_data;
-  vtable-priority = LT_DLLOADER_APPEND;
+  vtable-priority = LT_DLLOADER_PREPEND;
 }
 
   if (vtable  (vtable-dlloader_data != loader_data))

Re: HEAD: static tests

[Ralf Wildenhues]

Hi Gary,

* Gary V. Vaughan wrote on Wed, Feb 22, 2006 at 01:49:10PM CET:
 Ralf Wildenhues wrote:

[ tests for static linking ]

 Okay, thanks for the explanation.  I had forgotten much of the history :-(
 In light of these facts, then yes, please do commit this patch.

Done now.  Sorry for the long delay.

Cheers,
Ralf