[FFmpeg-devel] [PATCH] fate/ffprobe: add some stream metadata.

[Stefano Sabatini]

On date Thursday 2012-09-27 09:50:58 +0200, Clément Bœsch encoded:
> From: Clément Bœsch <clement.boesch at smartjog.com>
> 
> ---
>  tests/Makefile                 |    2 ++
>  tests/ref/fate/ffprobe_compact |   62 ++++++++++++++++-----------------
>  tests/ref/fate/ffprobe_csv     |   62 ++++++++++++++++-----------------
>  tests/ref/fate/ffprobe_default |   67 ++++++++++++++++++-----------------
>  tests/ref/fate/ffprobe_flat    |   67 ++++++++++++++++++-----------------
>  tests/ref/fate/ffprobe_ini     |   71 ++++++++++++++++++++-----------------
>  tests/ref/fate/ffprobe_json    |   75 ++++++++++++++++++++++------------------
>  tests/ref/fate/ffprobe_xml     |   67 ++++++++++++++++++-----------------
>  8 files changed, 250 insertions(+), 223 deletions(-)
> 
> diff --git a/tests/Makefile b/tests/Makefile
> index 68620006..f13bc7e 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -34,6 +34,8 @@ tests/data/ffprobe-test.nut: ffmpeg$(HOSTEXESUF) | tests/data
>  	$(M)./$< \
>          -f lavfi -i "aevalsrc=sin(400*PI*2*t)::d=0.125[out0]; testsrc=d=0.125[out1]; testsrc=s=100x100:d=0.125[out2]" \
>          -f ffmetadata -i $(SRC_PATH)/tests/test.ffmeta \

> +        -metadata:s:1 title=foobar -metadata:s:1 duration_ts=field-and-tags-conflict-attempt \
> +        -metadata:s:0 E=MC2 \

I'd prefer to add these to test.ffmeta and use the -map_metadata
option.

This way we test also the ffmetadata demuxer, -map_metadata and we
keep all the metadata things in a single file, which looks more
consistent (and workaround Windows commandline Unicode glitches).

Should be good otherwise, thanks.
-- 
FFmpeg = Friendly and Fundamentalist Meaningful Purposeless Erratic Gorilla