Re: [Mlt-devel] [mltframework/mlt] 064bb7: Increase libavutil version required to use PIX_FMT...

[Dan D. <da...@de...>]

On Mon, Aug 13, 2012 at 9:10 PM, GitHub <no...@gi...> wrote:
>   Branch: refs/heads/master
>   Home:   https://github.com/mltframework/mlt
>   Commit: 064bb789d7bc0778d7324279460f63c9da84ec4a
>       https://github.com/mltframework/mlt/commit/064bb789d7bc0778d7324279460f63c9da84ec4a
>   Author: Brian Matherly <pez...@ya...>
>   Date:   2012-08-13 (Mon, 13 Aug 2012)
>
>   Changed paths:
>     M src/modules/avformat/producer_avformat.c
>
>   Log Message:
>   -----------
>   Increase libavutil version required to use PIX_FMT_YUVA444P.
>
> PIX_FMT_YUVA444P was added to ffmpeg in version 51.35.101 - and so the precompiler check was correctly set to that version. However, it has not yet been added to libav. And libav recently increased the libavutil version to 51.38.0. This causes a compilation error against libav master because the precompiler check passes, but PIX_FMT_YUVA444P is not defined.
>
> The current libavutil versions are:
>  * ffmpeg master: 51.69.100
>  * ffmpeg 0.11:   51.54.100
>  * libav  master: 51.38.0
>
> This commit sets the precompiler to check against version 51.54.0. This will allow PIX_FMT_YUVA444P to be used when compiled against both ffmpeg master and ffmpeg 0.11 - while avoiding errors when compiling against libav master. However, if libav keeps incrementing the version without adding PIX_FMT_YUVA444P, this number may have to be increased again in the future.
>

We can identify ffmpeg vs libav with the following:

#if defined(FFUDIV)

Yes, it is ugly, but so is the fork, and it has been working for a
couple of months now.

-- 
+-DRD-+