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

[Dave Yeo]

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.
Dave