[MPlayer-dev-eng] [PATCH] Win32 loader support for OS/2

[KO Myung-Hun]

Hi/2.

Dave Yeo wrote:
> On 02/08/10 06:18 am, KO Myung-Hun wrote:
>> Hi/2.
>>
>> Diego Biurrun wrote:
>>> On Mon, Feb 08, 2010 at 09:37:01PM +0900, KO Myung-Hun wrote:
>>>> Hi/2.
>>>> Diego Biurrun wrote:
>>>>> On Fri, Feb 05, 2010 at 08:53:25PM +0900, KO Myung-Hun wrote:
>>>>>>
>>>>>> This patch adds Win32 loader support for OS/2.
>>>>>>
>>>>>> --- loader/module.c (revision 30512)
>>>>>> +++ loader/module.c (working copy)
>>>>>> @@ -744,6 +744,50 @@
>>>>>>
>>>>>> +#ifdef __OS2__
>>>>>> +uint32_t _System DosQueryMem(void *, uint32_t *, uint32_t *);
>>>>>
>>>>> Aren't you simply missing a header #include?
>>>>
>>>> It causes type clash between OS/2 and wine.
>>>
>>> Elaborate please. It still sounds like there is a better solution.
>>>
>>
>> Ok. Seeing is believing.
>>
>> I attach the error messages.
>
> I get most of these same errors with just your patch applied, from
> pthread.h including os2.h. Disabling pthreads also disables the win32
> loader in configure and the compile ends with a bunch of missing
> pthread_mutex_* related missing symbols from libdvdnav.

You should use the latest pthread from Yuri to which my patch is applied 
to solve this problem.

Requirements... part in readme.eng describes this.

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.0
Under OS/2 Warp 4 for Korean with FixPak #15
On Intel Core2Duo T5500 1.66 GHz with 1 GB RAM

Korean OS/2 User Community : http://www.ecomstation.co.kr