[FFmpeg-devel] [libav-devel] [PATCH 3/6] avutil: delay removal of the PIX_FMT_* flags

[Ronald S. Bultje]

Hi,

On Sun, Aug 9, 2015 at 11:54 AM, Andreas Cadhalpun <
andreas.cadhalpun at googlemail.com> wrote:

> Hi,
>
> On 09.08.2015 12:57, Ronald S. Bultje wrote:
> > Yeah, I'm with this. Andreas, the correct fix is to update applications,
> > even if that means vendor-specific patches in Debian. These are
> > exceptionally trivial patches that you can generate using fairly trivial
> > sed scripting.
>
> The problem is not that creating patches for these two API changes was
> difficult, but that these affect the majority of API users.
>
> > The same goes for other easily scriptable changes. These APIs are gone
> and
> > I don't want them back.
>
> What is your problem with keeping the trivial compatibility layers for
> another year?
>

We've decided several years ago that this API will go. This API is gone
now. Good riddance. We're going to make pancakes after breaking eggs. It's
going to be yummy. I'll help you decreasing the pain caused by the breaking
of the eggs. That's life.


> > Scriptable API changes are out and stay out. Just script a patch in
> Debian.
>
> I can help you scripting it together if you can't figure it out yourself
> > (something like find . -name *.[ch] -exec sed -i -e
> > 's|PIX_FMT_|AV_PIX_FMT|g' {} \; and then a similar version for this
> change)
> > should do the trick.
>
> This script will miss some cases (*.cpp) and wreck all sorts of havoc
> (AV_AV_PIX_FMT_*). But don't waste your time improving this script.
>
> If you want to help, please document how avpicture_deinterlace
> can be replaced in practice, that is in a project that doesn't use
> libavfilter yet.


OK. Do you want a documented example on something like stackoverflow, or a
deinterlace-example.c file, or something on the wiki?

Ronald