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-requ...@ffmpeg.org with subject "unsubscribe".
