Re: [PATCH] ltdl: set libltdl_cv_need_uscore to yes on OS/2

[Gary V. Vaughan]

Hi!

> On 27 Nov 2014, at 02:47, KO Myung-Hun <address@hidden> wrote:
> 
> Hi/2.
> 
> Gary V. Vaughan wrote:
>> Hi,
>> 
>> Thanks for the report and the patch!
>> 
>>> On Nov 22, 2014, at 4:08 AM, KO Myung-Hun <address@hidden> wrote:
>>> 
>>> On OS/2, dlopen() does not support a program. So libltdl_cv_need_uscore
>>> is set to unknown, but dlsym() requires an underscore prefix. So set
>>> libltdl_cv_need_uscore to yes on OS/2 if lt_cv_sys_symbol_underscore is
>>> yes and libltdl_cv_need_uscore is unknown.
>> 
>> Actually, I think the real problem here is that LT_FUNC_DLSYM_USCORE is
>> making the bad assumption that dlsym() only requires a leading symbol
>> name underscore on machines where self dlopening works.
>> 
>> Better than your suggested patch, we should really be checking whether
>> dlsym of ordinary loadable module symbol names requires a leading underscore.
> 
> I agree.
> 
>> I pushed the core of a new macro that does exactly that to M4 master just
>> now.
>> 
>> Would you let me know whether this works correctly on OS2 for you please?
> 
> Of course. Unfortunately, however, it does not work. dlopen() in
> configure fails due to 'file not found'.

Thanks for checking.  Can you tell me why it fails (module is not compiled 
correctly;
path argument to dlopen() is wrong), and maybe suggest what would fix it, 
please?

Cheers,
-- 
Gary V. Vaughan (gary AT gnu DOT org)