Re: [Vxl-users] Installing VXL on OS X Mavericks

[Matt L. <mat...@ki...>]

David,

I used to update vidl to track ffmpeg API changes, but I haven't had time
to keep up with it in the past few years.  An older version of ffmpeg
should work.  We could use a volunteer to update vidl_ffmpeg, unfortunately
I don't think I have time to do it.

-Matt
On Oct 27, 2014 5:20 PM, "David Pfau" <dav...@gm...> wrote:

> ...or maybe reverting to an older version of FFMpeg would work.
>
> On Mon, Oct 27, 2014 at 5:19 PM, David Pfau <dav...@gm...> wrote:
>
>> Actually, looks like the flags were moved around quite a bit in FFMpeg:
>>
>> http://ffmpeg.org/pipermail/ffmpeg-cvslog/2012-March/047603.html
>> http://ffmpeg.org/pipermail/ffmpeg-cvslog/2012-March/047604.html
>>
>> So now CODEC_FLAG_QP_RD and CODEC_FLAG_CBP_RD no longer exist in
>> avcodec.h but were replaced with FF_MPV_FLAG_QP_RD and FF_MPV_FLAG_CBP_RD
>> in mpegvideo.h. And these flags are not in AVCodecContext::flags but in
>> MpegCodecContext::mpv_flags. So I'm not sure what the relevant change to
>> make is - I don't see an MpegCodecContext instance anywhere in
>> vidl_ffmpeg_ostream_v0.11.txx.
>>
>> Out of curiosity - if these changes were made to FFMpeg over 2 years ago,
>> how did this not come up until now?
>>
>> -David
>>
>> On Mon, Oct 27, 2014 at 1:09 PM, Brad King <bra...@ki...> wrote:
>>
>>> On 10/26/2014 06:01 PM, David Pfau wrote:
>>> >
>>> */Users/pfau/Downloads/vxl/core/vidl/vidl_ffmpeg_ostream_v0.11.txx:308:25:
>>> **error: **use of undeclared identifier 'CODEC_FLAG_QP_RD'*
>>> >
>>> >     video_enc->flags |= CODEC_FLAG_QP_RD;
>>> >
>>> > *                        ^*
>>> >
>>> >
>>> */Users/pfau/Downloads/vxl/core/vidl/vidl_ffmpeg_ostream_v0.11.txx:312:25:
>>> **error: **use of undeclared identifier 'CODEC_FLAG_CBP_RD'*
>>> >
>>> >     video_enc->flags |= CODEC_FLAG_CBP_RD;
>>>
>>> I expect that code needs a fix similar to one I did in VTK:
>>>
>>>  vtkFFMPEGWriter: Fix build with libavcodec55
>>>  http://vtk.org/gitweb?p=VTK.git;a=commitdiff;h=63b5bc77
>>>
>>> Version 55 of the libavcodec API adds an AV_ prefix to the CODEC_ID_
>>> constants.
>>>
>>> -Brad
>>>
>>>
>>
>