[FFmpeg-devel] [PATCH] avcodec: Add padding after the remaining AVFrames

[Michael Niedermayer]

On Tue, Mar 18, 2014 at 01:25:26PM +0100, wm4 wrote:
> On Tue, 18 Mar 2014 00:48:41 +0100
> Michael Niedermayer <michaelni at gmx.at> wrote:
> 
> > This limits ABI issues in case libavcodec is linked to a libavutil with larger AVFrame
> > Which can happen if they are shiped in seperate binary packages and libavutil is upgraded
> > 
> > A cleaner alternative would be to replace them by pointers but this would likely cause
> > a small speedloss
> 
> Better idea: call abort() if the libavutil linked version doesn't match
> the compiled version.

That would badly break distro packages and is near certain to get
ffmpeg kicked out of distros and i actually agree that a lib
calling abort() when one of its dependancies has a different minor
version but same soname ... is just broken.


> When will people (including distro maintainers
> and ffmpeg developers) realize that mismatching different libav*
> versions is just broken?

There are just 2 options really
First, each lib has its own ABI, API and version (this is what we
have now)
or
Second, libs have the same ABI, API and version, possibly linked into
the same .so file

So far noone has sent a patch to add optional support for the 2nd
case to our build system. And this isnt the first time this has
come up ...

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140318/a40de488/attachment.asc>