[FFmpeg-devel] [PATCH] avfilter: add LIBVMAF filter

Ashish Pratap Singh ashk43712 at gmail.com
Sun Jul 16 18:33:52 EEST 2017


On Sun, Jul 16, 2017 at 6:19 PM, Derek Buitenhuis <
derek.buitenhuis at gmail.com> wrote:

> On 7/16/2017 1:29 PM, Ronald S. Bultje wrote:
> > Pushed.
> >
> > Ronald
>
> I am going to quote myself here, because it was never address, was not even
> replied to, and is outright incorrect on some OSes:
>
> >> +enabled libvmaf           && { check_lib libvmaf "libvmaf.h"
> "compute_vmaf" -lvmaf -lstdc++ -lpthread -lm ||
> >> +                               die "ERROR: libvmaf must be installed";
> }
> >
> > This won't work on, for example, OS X, which uses libc++ instead of
> libstdc++.
> > It'll also add extra unneeded libs to the linker flags when libvmaf is
> built
> > as a hared library,
> >
> > The correct solution is for libvmaf to provide a proper pkg-config file.
>
> Students should learn to reply to reviews instead of sending new patches
> with no changelog and having them pushed anyway.
>
> - Derek
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

Hi, sorry for that.
I will open a pull request soon (to libvmaf) for the pkg-config file.


More information about the ffmpeg-devel mailing list