Re: [PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-08-12 Thread Peter O'Gorman
Ralf Wildenhues wrote: > * Bob Friesenhahn wrote on Tue, Aug 12, 2008 at 09:27:07PM CEST: >> On Tue, 12 Aug 2008, Ralf Wildenhues wrote: >>> OK to revert? Or, rather: does anybody have problems with the reversal? >> Temporarily or permanently? > > Ah, chucks. Incomplete research. I take the abo

Re: [PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-08-12 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Tue, Aug 12, 2008 at 09:27:07PM CEST: > On Tue, 12 Aug 2008, Ralf Wildenhues wrote: >> >> OK to revert? Or, rather: does anybody have problems with the reversal? > > Temporarily or permanently? Ah, chucks. Incomplete research. I take the above back, for now, and brood

Re: [PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-08-12 Thread Bob Friesenhahn
On Tue, 12 Aug 2008, Ralf Wildenhues wrote: This patch causes a testsuite failure on GNU/Linux. As a step toward stabilization, I would like to propose to revert the above patch. Its acceptance should rely on its test passing on at least more than one of Darwin, MinGW, GNU/Linux, and AIX. (D

Re: [PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-08-12 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Sat, Aug 02, 2008 at 12:51:36PM CEST: > * Gary V. Vaughan wrote on Tue, May 06, 2008 at 07:24:46PM CEST: > > * libltdl/m4/ltdl.m4 (LTDL_INIT): Check for a libltdl that > > provides lt_dladvise_preopen when deciding if installed libltdl > > is 'new enough'. > > * libltdl/l

Re: [PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-08-02 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Tue, May 06, 2008 at 07:24:46PM CEST: > * libltdl/m4/ltdl.m4 (LTDL_INIT): Check for a libltdl that > provides lt_dladvise_preopen when deciding if installed libltdl > is 'new enough'. > * libltdl/libltdl/lt__private.h (lt__advise): Add a new > is_preload flag. >

Re: [PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-05-07 Thread Gary V. Vaughan
On 6 May 2008, at 14:07, Ralf Wildenhues wrote: Hi Gary, Hallo Ralf, a quick nit review (I haven't looked thoroughly yet): Thanks. I'll fix these nits and repost an updated [PATCH 370 bis] shortly. I'm only commenting on the nits I disagree with here. Everything else will be addresse

Re: [PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-05-06 Thread Ralf Wildenhues
Hi Gary, a quick nit review (I haven't looked thoroughly yet): * Gary V. Vaughan wrote on Tue, May 06, 2008 at 07:24:46PM CEST: > > This makes it possible to build a libltdl client with some preloaded > modules, and limit lt_dlopenadvise to open only the preloaded modules. > --- a/NEWS > +++ b/

[PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-05-06 Thread Gary V. Vaughan
* libltdl/m4/ltdl.m4 (LTDL_INIT): Check for a libltdl that provides lt_dladvise_preopen when deciding if installed libltdl is 'new enough'. * libltdl/libltdl/lt__private.h (lt__advise): Add a new is_preload flag. * libltdl/ltdl.c (lt_dladvise_preload): New api call to set it. (try_dlopen): If it is