[MPlayer-dev-eng] [PATCH] Fix libdvdcss on OS/2

Ivan Kalvachev ikalvachev at gmail.com
Sat Mar 9 21:07:34 CET 2013


On 1/24/13, Diego Biurrun <diego at biurrun.de> wrote:
> On Wed, Jan 23, 2013 at 02:39:32PM -0800, Dave Yeo wrote:
>> On 01/23/13 08:06 am, Diego Biurrun wrote:
>> >>>>[more errors]
>> >>>>>  >>
>> >>>>>>  >>>  And does upstream libdvdcss work?
>> >>>>>  >>
>> >>>>>  >>  Yes, the direct_h test includes win.h
>> >>>>>  >>
>> >>>>>>  >>>  In any case patches to libdvdcss should be submitted
>> >>>>>> upstream.
>> >>>>>  >>
>> >>>>>  >>  Or don't unconditionally include direct.h. It is only used in
>> >>>>>  >>  dvdcss.c and libavformat/os_support.h where it is wrapped in a
>> >>>>>  >>  #ifdef _WIN32.
>> >>>>>  >>  I'll look into it more after work
>> >>>>  >
>> >>>>  >  Please look into upstream libdvdcss and patch that, MPlayer can
>> >>>> later
>> >>>>  >  pick up the fix from there.
>> >>>>  >
>> >>>
>> >>>  I've confirmed that the upstream has no problem.
>> >>>
>> >>>  This is a problem of MPlayer configure.
>> >>>
>> >>>  I attach a patch.
>> >Fixed differently.
>>
>> Configure still finds direct.h on OS/2 and therefore defines
>> HAVE_DIRECT_H and fails on the Windows function in libdvdcss.c
>> Dave
>> http://svn.netlabs.org/libc/browser/branches/libc-0.6/src/emx/include/direct.h
>
> That needs to be fixed in libdvdcss.  You sent a patch to them
> already, just be a little bit patient for the response.

It's been almost 2 and half months and MPlayer dvdcss have not been fixed since.

Dave, I have committed the patch you have sent to libdvdcss.

It should hold until Diego merges the new libdvdcss.


More information about the MPlayer-dev-eng mailing list