Re: [FFmpeg-devel] [PATCH] cmdutils: Do not unconditionally define functions using avdevice's API

[Derek Buitenhuis]

On 2/8/2018 4:34 PM, James Almer wrote:
> I just reverted that commit, but in any case this patch needs to be
> taken into account for when (and if) it's reapplied.

Sounds good.

- Derek
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] cmdutils: Do not unconditionally define functions using avdevice's API

[James Almer]

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 
> ---
> 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

[FFmpeg-devel] [PATCH] cmdutils: Do not unconditionally define functions using avdevice's API

[Derek Buitenhuis]

avdevice is not a hard requiremnet for any of the fftools, and this was
broken in cdc78058c78dfa4966758a342acd2c1f3b282c46.

Fixes build with --disable-avdevice.

Signed-off-by: Derek Buitenhuis 
---
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;
-- 
1.8.3.1

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel