[FFmpeg-devel] [PATCH] Modular test sytem

[Diego Biurrun]

On Sat, Dec 26, 2009 at 02:01:07AM +0100, Stefano Sabatini wrote:
> Sample patch attached, which I tested it with lavf (I created a
> tests/lavf.regression.refd/ dir, and I put a file for each test with
> the corresponding line taken from lavf.regression.ref).
> 
> With the patch applied it is possible to edit the regression.list file
> (eliminting or commenting in the corresponding line) for disabling
> tests.

I'm not sure I understand the point of this.

> --- ffmpeg.orig/Makefile	2009-12-26 01:15:08.000000000 +0100
> +++ ffmpeg/Makefile	2009-12-26 02:00:43.000000000 +0100
> @@ -156,6 +157,13 @@
>  
> +$(CODEC_VSYNTH1_REFFILE) = tests/codec.regression.list
> +	cat $(patsubst regtest-%,$(SRC_PATH)/tests/codec-vsynth1.regression.refd/%, $(LAVF_TESTS)) > $@
> +$(CODEC_VSYNTH2_REFFILE) = tests/codec.regression.list
> +	cat $(patsubst regtest-%,$(SRC_PATH)/tests/codec-vsynth2.regression.refd/%, $(LAVF_TESTS)) > $@
> +$(LAVF_REFFILE): tests/lavf.regression.list
> +	cat $(patsubst regtest-%,$(SRC_PATH)/tests/lavf.regression.refd/%,          $(LAVF_TESTS)) > $@

Is the ".refd" instead of ".ref" intentional?  If yes, I don't see where
that directory is created and am confused.

Diego