[FFmpeg-devel] [PATCH 6/8] fate: introduce subtitles.mak and move SubRip test in it.

[Michael Niedermayer]

On Fri, Jun 15, 2012 at 07:29:13PM +0200, Clément Bœsch wrote:
> ---
>  tests/Makefile           |    1 +
>  tests/fate/subtitles.mak |    7 +++++++
>  tests/fate/video.mak     |    3 ---
>  3 files changed, 8 insertions(+), 3 deletions(-)
>  create mode 100644 tests/fate/subtitles.mak
> 
> diff --git a/tests/Makefile b/tests/Makefile
> index d91c079..266901c 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -72,6 +72,7 @@ include $(SRC_PATH)/tests/fate/qt.mak
>  include $(SRC_PATH)/tests/fate/qtrle.mak
>  include $(SRC_PATH)/tests/fate/real.mak
>  include $(SRC_PATH)/tests/fate/screen.mak
> +include $(SRC_PATH)/tests/fate/subtitles.mak
>  include $(SRC_PATH)/tests/fate/utvideo.mak
>  include $(SRC_PATH)/tests/fate/video.mak
>  include $(SRC_PATH)/tests/fate/voice.mak
> diff --git a/tests/fate/subtitles.mak b/tests/fate/subtitles.mak
> new file mode 100644
> index 0000000..6be579d
> --- /dev/null
> +++ b/tests/fate/subtitles.mak
> @@ -0,0 +1,7 @@
> +FATE_SUBTITLES += fate-sub-srt
> +fate-sub-srt: CMD = md5 -i $(SAMPLES)/sub/SubRip_capability_tester.srt -f ass
> +

> +FATE_SUBTITLES += $(FATE_SUBTITLES-yes)

why is this needed ?

otherwise LGTM

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120616/80e5c834/attachment.asc>