Anton Khirnov <[email protected]> writes:

> ---
>  tests/Makefile           |    1 +
>  tests/fate/cover_art.mak |    7 +++++++
>  2 files changed, 8 insertions(+)
>  create mode 100644 tests/fate/cover_art.mak
>
> diff --git a/tests/Makefile b/tests/Makefile
> index 429762b..6203f48 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -36,6 +36,7 @@ include $(SRC_PATH)/tests/fate/atrac.mak
>  include $(SRC_PATH)/tests/fate/audio.mak
>  include $(SRC_PATH)/tests/fate/bmp.mak
>  include $(SRC_PATH)/tests/fate/cdxl.mak
> +include $(SRC_PATH)/tests/fate/cover_art.mak
>  include $(SRC_PATH)/tests/fate/dct.mak
>  include $(SRC_PATH)/tests/fate/demux.mak
>  include $(SRC_PATH)/tests/fate/dfa.mak
> diff --git a/tests/fate/cover_art.mak b/tests/fate/cover_art.mak
> new file mode 100644
> index 0000000..5d2d81e
> --- /dev/null
> +++ b/tests/fate/cover_art.mak
> @@ -0,0 +1,7 @@
> +FATE_COVER_ART += fate-cover_art-m4a

Diego probably wants you to use hyphens rather than underscores, and
this case I'm inclined to agree.

> +fate-cover_art-m4a: CMD = md5 -i 
> $(SAMPLES)/cover_art/Owner-iTunes_9.0.3.15.m4a -an -c:v copy -f rawvideo

-c:v copy together with -f rawvideo looks weird, but I guess that's how
it's done.

> +fate-cover_art-m4a: REF = 08ba70a3b594ff6345a93965e96a9d3e
> +
> +$(FATE_COVER_ART): CMP = oneline
> +FATE_SAMPLES_AVCONV += $(FATE_COVER_ART)
> +fate-cover_art: $(FATE_COVER_ART)
> -- 

LGTM otherwise.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to