[MPlayer-users] Build error MinGW/VCD

[Josh]

On 8/29/2015 15:37, Roberto Togni wrote:
> On Sat, 29 Aug 2015 13:40:24 -0400
> Josh <joshf87 at live.com> wrote:
>
>> I'm unable to build with VCD enabled in the latest SVN under
>> MinGW-w64/GCC 4.9.3. I am using libcdio 0.93/paranoia 10.2+0.93. I was
>> using 0.83 thinking that was the problem but updating to 0.93 didn't
>> change anything. Here are the log files if they are needed:
>> http://mplayerwin.sourceforge.net/misc/mpl-bin-r37465/buildinfo/
>>
>> stream/stream_vcd.c: In function 'close_s':
>> stream/stream_vcd.c:131:3: error: implicit declaration of function
>> 'cdio_destroy' [-Werror=implicit-function-declaration]
>>      cdio_destroy(vcd->cdio);
>>      ^
>> stream/stream_vcd.c:131:19: error: 'mp_vcd_priv_t' has no member named
>> 'cdio'
>>      cdio_destroy(vcd->cdio);
>>                      ^
>> stream/stream_vcd.c: In function 'open_s':
>> stream/stream_vcd.c:154:3: error: unknown type name 'CdIo'
>>      CdIo *cdio;
>>      ^
>> stream/stream_vcd.c:211:8: error: 'mp_vcd_priv_t' has no member named 'cdio'
>>        vcd->cdio = cdio;
>>           ^
>> cc1.exe: some warnings being treated as errors
>> Makefile:731: recipe for target 'stream/stream_vcd.o' failed
>> make: *** [stream/stream_vcd.o] Error 1
>>
> The attached patch may fix it for you, but it's probably not the best
> fix (you lose the new functionalities about playing video CDs with
> libcdio, even if I have no idea if that may work in Windows).
>
>
> Please test, if it works I will commit it while waiting for a bet fix.
>
> Ciao,
>   Roberto
>
>
>
> _______________________________________________
> MPlayer-users mailing list
> MPlayer-users at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/mplayer-users
>

Compiled successfully.

I have no idea if VCDs even work since I don't have any but it is 
autodetected by configure as being supported, after fixing header paths 
which aren't correct for MinGW-w64 anyway:

http://mplayerwin.sourceforge.net/misc/0012-vcd-support-on-MinGW-w64.patch
http://mplayerwin.sourceforge.net/misc/0011-stream_cddb-fix-include-path-on-MinGW-w64.patch