[FFmpeg-devel] [PATCH] checkasm: Add a "run-checkasm" make target

[Martin Storsjö]

On Wed, 14 Feb 2024, Martin Storsjö wrote:

> Contrary to the existing "fate-checkasm", this always prints the
> tool output, and runs all tests at once instead of splitting it up
> per target group. This is more useful when the user expects to
> look directly at the tool output, instead of being part of a full
> fate run.
>
> (On failure with the regular "make fate-checkasm" targets, none of
> the tool output is printed, but stored in files. If run with reporting
> set up to the FATE website, the individual failures are uploaded there,
> but if it is run in some sort of other CI setup, the intermediate files
> might not be available afterwards for inspection.)
> ---
> tests/checkasm/Makefile | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/tests/checkasm/Makefile b/tests/checkasm/Makefile
> index 3562acb2b2..3af42a679b 100644
> --- a/tests/checkasm/Makefile
> +++ b/tests/checkasm/Makefile
> @@ -91,6 +91,10 @@ CHECKASM := tests/checkasm/checkasm$(EXESUF)
> $(CHECKASM): $(CHECKASMOBJS) $(FF_STATIC_DEP_LIBS)
> 	$(LD) $(LDFLAGS) $(LDEXEFLAGS) $(LD_O) $(CHECKASMOBJS) $(FF_STATIC_DEP_LIBS) $(EXTRALIBS-avcodec) $(EXTRALIBS-avfilter) $(EXTRALIBS-avformat) $(EXTRALIBS-avutil) $(EXTRALIBS-swresample) $(EXTRALIBS)
>
> +run-checkasm: $(CHECKASM)
> +run-checkasm:
> +	$(TARGET_EXEC) $(TARGET_PATH)/$(CHECKASM)

I've amended this locally with a $(Q) at the start, to silence the 
executed command, unless executed with V=1.

I'll push this patch later today if there aren't any objections.

// Martin