[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] ltdl: set libltdl_cv_need_uscore to yes on OS/2
From: |
Gary V. Vaughan |
Subject: |
Re: [PATCH] ltdl: set libltdl_cv_need_uscore to yes on OS/2 |
Date: |
Thu, 27 Nov 2014 03:21:10 +0000 |
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)