On Sun, 1 Dec 2024, Alexander Strasser via ffmpeg-devel wrote:
Looks good and useful to me.
Thanks!
Just a few ideas follow:
diff --git a/doc/build_system.txt b/doc/build_system.txt
index 0b1b0c2054..91c7a5e9af 100644
--- a/doc/build_system.txt
+++ b/doc/build_system.txt
@@ -30,6 +30,12 @@ fate
fate-list
List all fate/regression test targets.
+fate-list-failing
+ List the fate tests that failed the last time they were executed.
+
+fate-clear-results
+ Remove the test results from previous test executions.
+
Would it be better to use the same description as int fate.texi ?
Sure, I can add that extra parenthesis.
diff --git a/tests/Makefile b/tests/Makefile
index 9b70145015..adb0799328 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -313,6 +313,12 @@ $(FATE): $(FATE_UTILS:%=tests/%$(HOSTEXESUF)) |
$(FATE_OUTDIRS)
fate-list:
@printf '%s\n' $(sort $(FATE))
+fate-list-failing:
+ @cat tests/data/fate/*.rep | cut -f 1-2 -d : | grep -v :0 | sed
's/:.*//;s/^/fate-/'
+
+fate-clear-results:
+ @rm -f tests/data/fate/*.rep
+
coverage.info: TAG = LCOV
coverage.info:
$(M)lcov -q -d $(CURDIR) -b $(patsubst src%,./,$(SRC_LINK)) --capture |
\
--
Maybe the attached patch would make sense on top of your changes?
Nice, thanks. I'm not very familiar with awk so I seldom write such things
myself, but this does indeed seem neater.
As a second question, do you have any opinion on the fate target names?
Should the second be "fate-clear-failing" to match the list-failing one?
But technically it's not only about clearing the failing ones but clearing
all results...
// 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".