[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: feedback on libltdl changes
From: |
Gary V . Vaughan |
Subject: |
Re: feedback on libltdl changes |
Date: |
Sat, 17 Jul 2004 16:07:28 +0100 |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Hi Greg,
Thanks again for the feedback.
I'm just off for a week of camping and Tae Kwon-do, so I don't have
time to
look into these now :-(
I'll postpone the alpha release until these are done, if any intrepid
souls
would like to send a fix to address@hidden for my return a
week on
Monday, I'll put the release out then :-)
Cheers,
Gary.
On 16 Jul 2004, at 19:28, Greg Eisenhauer wrote:
Hi Gary,
I dropped the current libtool CVS head on my fedora box to give is a
go. I found two issues, one easy and one not so easy (at least for
me).
First the easy issue. I think there's a typo in the last line of
libtool/libltdl/loaders/Makefile.am. In particular in the recursive
invocation of make to build ../libdlloader.la
../libdlloader.la:
cd ..; $(MAKE) $(MAKEFLAGS) libdlloader.la
I changed $(MAKEFLAGS) to $(AM_MAKEFLAGS).
For the second issue, the proper fix is less clear to me. Basically
mdemo fails to link on Fedora because it is missing "-ldl" on the link
line,
resulting in undefined symbols for dlopen, etc. In libtool-1.5.6, the
-ldl
seems to come about because it is in dependency_libs in libm.la. In
turn,
it seems to have gotten *there* because it is in dependency_libs in
libltdl.la.
However, in the current head release, -ldl does appear in
dependency_libs in loaders/dlopen.la, but it doesn't end up in the
dependency_libs when libltdl.la is built. (So it isn't in libm.la,
therefor
isn't specified in the mdemo link, and the link fails. I guess it
isn't
clear to me if there's something that needs special-case handling here
and
I'm not sure I could produce a surgical patch if there is. Any
thoughts?
greg
-----------------------------------------------------------------------
------
greg eisenhauer address@hidden (404)894-3227
http://www.cc.gatech.edu/~eisen/
College of Computing, Georgia Institute of Technology, Atlanta, GA
30332
- --
Gary V. Vaughan ())_. address@hidden,gnu.org}
Research Scientist ( '/ http://tkd.kicks-ass.net
GNU Hacker / )= http://www.gnu.org/software/libtool
Technical Author `(_~)_ http://sources.redhat.com/autobook
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2 (Darwin)
iD8DBQFA+UC0FRMICSmD1gYRArKvAKCyh1QwZN78wt3zTp1gy7qBZfs5lACeNhca
dKPOaR1T0wV7P9Fjmr2ztAs=
=IwLA
-----END PGP SIGNATURE-----
- Re: feedback on libltdl changes,
Gary V . Vaughan <=