[FFmpeg-devel] [PATCH] fate: warn if SAMPLES is not specified.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Thu Dec 22 23:42:50 CET 2011


On Thu, Dec 22, 2011 at 04:23:05PM +0100, Clément Bœsch wrote:
> ---
>  tests/Makefile |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/Makefile b/tests/Makefile
> index b44db7d..d825cd8 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -108,9 +108,13 @@ fate-seek:   $(FATE_SEEK)
>  
>  ifdef SAMPLES
>  FATE += $(FATE_TESTS) $(FATE_TESTS-yes)
> +fate: $(FATE)
>  fate-rsync:
>  	rsync -vaLW --timeout=60 --contimeout=60 rsync://fate.ffmpeg.org/fate-suite/ $(SAMPLES)
>  else
> +fate-samples-warning:
> +	@echo "warning: only a subset of the fate tests will be run because SAMPLES is not specified"
> +fate: fate-samples-warning $(FATE)
>  fate-rsync:
>  	@echo "use 'make fate-rsync SAMPLES=/path/to/samples' to sync the fate suite"
>  $(FATE_TESTS):
> @@ -121,8 +125,6 @@ FATE_UTILS = base64 tiny_psnr
>  
>  TOOL = ffmpeg
>  
> -fate: $(FATE)
> -

Hm, I don't think that should be necessary to change, just specifying
the additional fate-samples-warning dependency should work.
However particularly with -j <n> it can be printed whenever.
(I am not completely sure about the ':' vs. '::' syntax, might be you
have to switch to using ::, but at least duplicating it completely
should not be necessary).
Alternatively you could just do a
FATE += fate-samples-warning
but that would probably print it at the end - whether that is better or
worse would be up to discussion.


More information about the ffmpeg-devel mailing list