On 2016-11-24 19:32:54 +0100, Diego Biurrun wrote:
> On Thu, Nov 24, 2016 at 06:44:35PM +0100, Janne Grunau wrote:
> > On 2016-11-24 17:23:49 +0100, Diego Biurrun wrote:
> > > --- a/configure
> > > +++ b/configure
> > > @@ -2440,6 +2441,7 @@ frei0r_src_filter_extralibs='$ldl'
> > >  hqdn3d_filter_deps="gpl"
> > >  interlace_filter_deps="gpl"
> > > +movie_filter_deps="avcodec avformat"
> > >  ocv_filter_deps="libopencv"
> > >  resample_filter_deps="avresample"
> > > --- a/libavfilter/vsrc_movie.c
> > > +++ b/libavfilter/vsrc_movie.c
> > > @@ -35,7 +35,11 @@
> > >  #include "libavutil/opt.h"
> > >  #include "libavutil/imgutils.h"
> > > +
> > > +#include "libavcodec/avcodec.h"
> > > +
> > >  #include "libavformat/avformat.h"
> > > +
> > >  #include "avfilter.h"
> > >  #include "formats.h"
> > 
> > unrelated? At least a little surprising given the commit message
> 
> Sort of related. I looked into the file and noticed that it was missing
> the avcodec.h #include. Moving the addition of that #include to a separate
> patch felt a bit silly. I can drop this hunk if you prefer...

just mention it in the commit message. patch ok with that

Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to