[FFmpeg-devel] [PATCH] Revert "Merge commit '3bc2e89c76e88ae6f1fd5287e0b11abcfc3c601c'"

James Almer jamrial at gmail.com
Wed Dec 18 03:26:28 CET 2013


On 17/12/13 11:10 PM, Michael Niedermayer wrote:
> On Mon, Dec 16, 2013 at 11:54:40PM +0100, Michael Niedermayer wrote:
>> This reverts commit 792845e436ddc5f3ce11ba02535a3372919061f2, reversing
>> changes made to 1d6666a6b828cca6c6b355a9b62f25607b1ced08.
>>
>> Bumping libavutil requires all libraries that use libavutil to have their
>> major version bumped (yes breakage has been confirmed this is not a hypotheses)
>>
>> One case of breakage is due to new types being added to AVOptions and
>> applications that linked to old libavutil and libswresample
>> then trying to use old libavutil (its soname changed so the old isnt updated)
>> and new swresample (its soame didnt change so it is updated)
>> the new swresample contains AVOption types that the old libavutil doesnt
>> know of thus the application attempting to access these avoptions
>> fails
>>
>> AVOptions are used by all libs so the issue can potentially happen with
>> any other lib, libswresample was just the first that showed the problem
>> ive not checked if the other libs are affected currently by the same issue
>> or not
>> ---
>>  doc/APIchanges      |    2 +-
>>  libavutil/opt.c     |    6 +++---
>>  libavutil/version.h |   40 ++++++++++++++++++++--------------------
>>  3 files changed, 24 insertions(+), 24 deletions(-)
> 
> applied (with micro ver bump and without FF_API changes)

>From the commit message 

> "The alternative of bumping all is still possible after this revert, if it
> turns out to be the preferred solution"

If the fork goes that way, then we should as well. It will save us and distros 
many headaches.


More information about the ffmpeg-devel mailing list