On 2/8/2018 12:21 PM, Derek Buitenhuis wrote: > avdevice is not a hard requiremnet for any of the fftools, and this was > broken in cdc78058c78dfa4966758a342acd2c1f3b282c46.
I just reverted that commit, but in any case this patch needs to be taken into account for when (and if) it's reapplied. > > Fixes build with --disable-avdevice. > > Signed-off-by: Derek Buitenhuis <derek.buitenh...@gmail.com> > --- > I see you guys can't decide on whether to revert or not, but in > the meantime, there is a fix for build breakage. Please don't > reply with something like "we can't push until we finish > bikeshedding over reverting or not". > > Quite a lot of people build without avdevice. > --- > fftools/cmdutils.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c > index 0b06ccc..4f482d6 100644 > --- a/fftools/cmdutils.c > +++ b/fftools/cmdutils.c > @@ -1281,9 +1281,11 @@ static int show_formats_devices(void *optctx, const > char *opt, const char *arg, > } } while(0) > > x(av_muxer_iterate, ofmt, muxdemuxers != SHOW_DEMUXERS && > !device_only); > - x(av_outdev_iterate, ofmt, muxdemuxers != SHOW_DEMUXERS); > x(av_demuxer_iterate, ifmt, muxdemuxers != SHOW_MUXERS && > !device_only); > +#if CONFIG_AVDEVICE > + x(av_outdev_iterate, ofmt, muxdemuxers != SHOW_DEMUXERS); > x(av_indev_iterate, ifmt, muxdemuxers != SHOW_MUXERS); > +#endif > #undef x > if (!name) > break; > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
