[MPlayer-dev-eng] [PATCH] Bluray support through libbluray

Benjamin Zores benjamin.zores at gmail.com
Sun Jul 4 09:35:52 CEST 2010



Le 4 juil. 2010 à 08:19, Ergzay <ergzay at gmail.com> a écrit :

>
> On Jul 3, 2010, at 7:09 PM, Reimar Döffinger wrote:
>
>> On Sat, Jul 03, 2010 at 10:47:56PM +0200, Benjamin Zores wrote:
>>> On Sat, Jul 3, 2010 at 10:14 PM, Dâniel Fraga <fragabr at gmail.com 
>>> > wrote:
>>>> On Sat, 3 Jul 2010 16:44:09 +0200
>>>> Benjamin Zores <ben at geexbox.org> wrote:
>>>>
>>>>> Done.
>>>>> See new patch.
>>>>
>>>>       Benjamin, thanks for the patch. I tried it with latest
>>>> libbluray from git repository and I got this:
>>>>
>>>> cc -MD -MP -Wstrict-prototypes -Wmissing-prototypes -Wundef - 
>>>> Wdisabled-optimization -Wno-pointer-sign -Wdeclaration-after- 
>>>> statement -std=gnu99 -Wall -Wno-switch -Wpointer-arith - 
>>>> Wredundant-decls -O4 -march=native -mtune=native -pipe -ffast- 
>>>> math -fomit-frame-pointer -Wstrict-prototypes -Wmissing- 
>>>> prototypes -Wundef -Wdisabled-optimization -Wno-pointer-sign - 
>>>> Wdeclaration-after-statement -std=gnu99 -Wall -Wno-switch - 
>>>> Wpointer-arith -Wredundant-decls -O4 -march=native -mtune=native - 
>>>> pipe -ffast-math -fomit-frame-pointer -D_LARGEFILE_SOURCE - 
>>>> D_FILE_OFFSET_BITS=64 -D_LARGEFILE64_SOURCE -Ilibdvdread4 -I.  - 
>>>> D_REENTRANT -I/usr/X11/include   -I/usr/local/include/artsc - 
>>>> pthread -I/usr/local/include/glib-2.0 -I/usr/local/lib/glib-2.0/ 
>>>> include  -I/usr/local/include/freetype2 -I/usr/local/include -I/ 
>>>> usr/local/include   -c -o stream/stream_bluray.o stream/ 
>>>> stream_bluray.c
>>>> stream/stream_bluray.c: In function 'bluray_stream_open':
>>>> stream/stream_bluray.c:105:5: error: 'BD_TITLE_INFO' undeclared  
>>>> (first use in this function)
>>>> stream/stream_bluray.c:105:5: note: each undeclared identifier is  
>>>> reported only once for each function it appears in
>>>> stream/stream_bluray.c:105:20: error: 'info' undeclared (first  
>>>> use in this function)
>>>> stream/stream_bluray.c:106:5: warning: ISO C90 forbids mixed  
>>>> declarations and code
>>>> stream/stream_bluray.c:148:24: error: 'ti' undeclared (first use  
>>>> in this function)
>>>> stream/stream_bluray.c:149:9: warning: ISO C90 forbids mixed  
>>>> declarations and code
>>>> make: *** [stream/stream_bluray.o] Error 1
>>>>
>>>>       Maybe it's trivial? Thanks.
>>>
>>> Yeah, replace the BD_ by BLURAY_
>>> I made libbluray guys change to BLURAY_* for consistency this
>>> afternoon and I have another patch for that.
>>
>> Should you maybe extend the configure check to make sure it is a  
>> version
>> we can use and possibly print some hint about which version is  
>> required?
>> I also always appreciate it when people double-check not to add any
>> warnings :-).
>> Otherwise I have no objections...
>
> On Mac OS X 10.6 here. I just tried a compile with the patch and it  
> fails on compile. Comes about because it can't find a malloc.h.  
> Probably some #ifdef needed.
>
> cc -MD -MP -Wstrict-prototypes -Wmissing-prototypes -Wundef - 
> Wdisabled-optimization -Wno-pointer-sign -Wdeclaration-after- 
> statement -std=gnu99 -Wall -Wno-switch -Wpointer-arith -Wredundant- 
> decls -O4 -march=core2 -mtune=core2 -pipe -ffast-math -fomit-frame- 
> pointer -Wstrict-prototypes -Wmissing-prototypes -Wundef -Wdisabled- 
> optimization -Wno-pointer-sign -Wdeclaration-after-statement - 
> std=gnu99 -Wall -Wno-switch -Wpointer-arith -Wredundant-decls -O4 - 
> march=core2 -mtune=core2 -pipe -ffast-math -fomit-frame-pointer - 
> D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE64_SOURCE - 
> Ilibdvdread4 -mdynamic-no-pic -falign-loops=16 -shared-libgcc -I.  - 
> DPIC  -I/usr/X11/include -I/usr/local/include/ -I/usr/local/include/ 
> SDL -D_GNU_SOURCE=1 -D_THREAD_SAFE -I/usr/X11/include/freetype2 -I/ 
> usr/X11/include -c -o stream/stream_bluray.o stream/stream_bluray.c
> stream/stream_bluray.c:31:20: error: malloc.h: No such file or  
> directory
> make: *** [stream/stream_bluray.o] Error 1

Can you try to send a patch to fix it on OSX ? I really can't test here.

Ben



>
> Matthew Lindner
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng



More information about the MPlayer-dev-eng mailing list