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

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

[Martin Storsjö]

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)
+
 checkasm: $(CHECKASM)
 
 testclean:: checkasmclean
-- 
2.39.3 (Apple Git-145)

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [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
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".