cygwin dlopening backends (was: FYI: 277-gary-rename-remaining-troublesome-ltdl-apis.diff)

2005-10-28 Thread Ralf Wildenhues
Hi Charles, Eric, * Charles Wilson wrote on Thu, Oct 27, 2005 at 01:45:05AM CEST: > Eric Blake wrote: > >> > >>Not quite. On a managed mount, cygwin_conv_to_full_win32_path > >>(searchname, wpath); correctly returns the munged underlying > >>Windows name, but earlier in the code, you appended a t

Re: FYI: 277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-10-26 Thread Charles Wilson
Eric Blake wrote: Not quite. On a managed mount, cygwin_conv_to_full_win32_path (searchname, wpath); correctly returns the munged underlying Windows name, but earlier in the code, you appended a trailing dot. Since a managed mount munges the trailing dot, you are now trying to load (for exampl

Re: FYI: 277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-10-26 Thread Eric Blake
> > Not quite. On a managed mount, cygwin_conv_to_full_win32_path > (searchname, wpath); correctly returns the munged underlying > Windows name, but earlier in the code, you appended a trailing > dot. Since a managed mount munges the trailing dot, you are > now trying to load (for example) "c:\c

Re: FYI: 277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-10-26 Thread Eric Blake
> Hi Eric, > > * Eric Blake wrote on Wed, Oct 26, 2005 at 05:11:57PM CEST: > > CVS head is currently broken on cygwin. It looks like the patch > > to rename lt_dlhandle_next to lt_dlhandle_iterate was incomplete. > > Yep. And we see me goofing up once more: loadlibrary.c:vm_open() > really _has

Re: FYI: 277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-10-26 Thread Eric Blake
Gary V. Vaughan gnu.org> writes: > -lt_dlhandle_next (lt_dlhandle place) > +lt_dlhandle_iterate (lt_dlinterface_id iface, lt_dlhandle place) >{ > lt__handle *handle = (lt__handle *) place; > + lt__interface_id *iterator = (lt__interface_id *) iface; > + > + assert (iface); /*

Re: FYI: 277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-10-26 Thread Ralf Wildenhues
Hi Eric, * Eric Blake wrote on Wed, Oct 26, 2005 at 05:11:57PM CEST: > CVS head is currently broken on cygwin. It looks like the patch > to rename lt_dlhandle_next to lt_dlhandle_iterate was incomplete. Yep. And we see me goofing up once more: loadlibrary.c:vm_open() really _has_ to iterate ove

Re: FYI: 277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-10-26 Thread Eric Blake
CVS head is currently broken on cygwin. It looks like the patch to rename lt_dlhandle_next to lt_dlhandle_iterate was incomplete. $ make CFLAGS='-g2 -Wall -Werror' ... make[2]: Entering directory `/home/eblake/libtool' depbase=`echo libltdl/loaders/loadlibrary.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$

FYI: 277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-10-26 Thread Gary V. Vaughan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Applied to HEAD. * looking for [EMAIL PROTECTED]/libtool--devo--1.0--patch-331 to compare with * comparing to [EMAIL PROTECTED]/libtool--devo--1.0--patch-331 M NEWS M doc/libtool.texi M libltdl/ltdl.c M libltdl/ltdl.h M ChangeLog

FYI: 277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-10-26 Thread Gary V. Vaughan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Applied to HEAD. * looking for [EMAIL PROTECTED]/libtool--devo--1.0--patch-331 to compare with * comparing to [EMAIL PROTECTED]/libtool--devo--1.0--patch-331 M NEWS M doc/libtool.texi M libltdl/ltdl.c M libltdl/ltdl.h M ChangeLog

Re: [patch 01/19] 277-gary-rename-remaining-troublesome-ltdl-apis.diff Queue

2005-10-25 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Tue, Oct 25, 2005 at 10:36:00AM CEST: > Ralf Wildenhues wrote: > > > >Please also apply the then-freed part of your patch queue, > > Before 277, I have (see my patch queue status mail from last Tue): Gah, I've been unaware of the fact that your patch queue ma

Re: [patch 01/19] 277-gary-rename-remaining-troublesome-ltdl-apis.diff Queue

2005-10-25 Thread Gary V. Vaughan
Ralf Wildenhues wrote: Hi Gary, Hallo Ralf, Thanks for the review! * Gary V. Vaughan wrote on Mon, Oct 10, 2005 at 12:26:25PM CEST: Index: libtool--devo--1.0/ChangeLog from Gary V. Vaughan <[EMAIL PROTECTED]> * libltdl/ltdl.h, libltdl/ltdl.c (lt_dlhandle_first): Removed. *

Re: [patch 01/19] 277-gary-rename-remaining-troublesome-ltdl-apis.diff Queue

2005-10-24 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Mon, Oct 10, 2005 at 12:26:25PM CEST: > ChangeLog|0 > NEWS |9 +- > doc/libtool.texi | 176 > +-- > libltdl/ltdl.c | 88 ++- > libltdl/ltdl.h | 1

[patch 01/19] 277-gary-rename-remaining-troublesome-ltdl-apis.diff Queue

2005-10-10 Thread Gary V. Vaughan
ChangeLog|0 NEWS |9 +- doc/libtool.texi | 176 +-- libltdl/ltdl.c | 88 ++- libltdl/ltdl.h | 10 +-- 4 files changed, 114 insertions(+), 169 deletions(-) Index: libtool--devo--1.0/C

Re: 277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-09-27 Thread Gary V. Vaughan
Ralf Wildenhues wrote: Hi Gary, Hallo Ralf, * Gary V. Vaughan wrote on Sun, Sep 18, 2005 at 02:30:03AM CEST: This fixes the remaining holes that allowed libltdl clients to stumble across each others' loaded modules. Of course, there is nothing to prevent clients from registering a dumb inte

Re: 277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-09-25 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Sun, Sep 18, 2005 at 02:30:03AM CEST: > Okay to commit? > > This fixes the remaining holes that allowed libltdl clients to stumble > across each others' loaded modules. Of course, there is nothing to > prevent clients from registering a dumb interface validato

277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-09-18 Thread Gary V. Vaughan
Repost. This one applies cleanly over 275 and 276, posted earlier. ChangeLog| 11 +++ NEWS |9 +- doc/libtool.texi | 176 +-- libltdl/ltdl.c | 84 +- libltdl/ltdl.h | 10 +-- 5 files ch

277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-09-18 Thread Gary V. Vaughan
D'oh, stupid typo in the last post. Here is the correct patch: ChangeLog| 11 +++ NEWS |9 +- doc/libtool.texi | 176 +-- libltdl/ltdl.c | 88 ++- libltdl/ltdl.h | 10 +-- 5 files chang

277-gary-rename-remaining-troublesome-ltdl-apis.diff

2005-09-17 Thread Gary V. Vaughan
Okay to commit? This fixes the remaining holes that allowed libltdl clients to stumble across each others' loaded modules. Of course, there is nothing to prevent clients from registering a dumb interface validator that returns other clients' modules... hopefully the extra stuff I've added to the