Generally I would not be opposed to something like this being tied to an
option in .config, so users can choose whether or not they wish to disable
retests (I personally retest frequently). Of course the default behavior
there would need to be allowing them (unchanged), but you can add
disable_retest=1 or something to disable them. Instead of making this a
new script, incorporate it into runtests.sh with the .config checking and
you can use $testset to verify. You should probably also use $logdir in
case people override this locally.
Thanks,
Justin
On Mon, Apr 4, 2016 at 12:45 PM, Charles-Antoine Couret <
renault(a)fedoraproject.org> wrote:
From dda596d968b7e09d821d842129b38a2771f63ebb Mon Sep 17 00:00:00
2001
From: Charles-Antoine Couret <charles-antoine.couret(a)nexvision.fr>
Date: Mon, 4 Apr 2016 10:34:33 +0200
Subject: [PATCH] Add script shell wrapper to check if the wanted test was
not
tested for current version.
It should avoid to verify that manually before executing the test
(or to execute twice the test which is useless).
Signed-off-by: Charles-Antoine Couret <charles-antoine.couret(a)nexvision.fr
>
---
kernel-tests.sh | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
create mode 100755 kernel-tests.sh
diff --git a/kernel-tests.sh b/kernel-tests.sh
new file mode 100755
index 0000000..397264c
--- /dev/null
+++ b/kernel-tests.sh
@@ -0,0 +1,22 @@
+#!/bin/sh
+
+if [ -z $1 ]; then
+ echo "You must provide a test name."
+ exit 1
+fi
+
+TEST=$1
+CURRENT_VERSION=$(uname -r)
+
+# Check if wanted test has been executed with current kernel version
+for file in logs/*; do
+ VERSION_TESTED=$(cat $file | sed -n 3p | cut -d ' ' -f 2)
+ TEST_EXECUTED=$(cat $file | sed -n 2p | cut -d ' ' -f 3)
+
+ if [ "$VERSION_TESTED" == "$CURRENT_VERSION" -a
"$TEST_EXECUTED" ==
"$TEST" ]; then
+ echo "The current kernel was already tested with this test,
abort."
+ exit 0
+ fi
+done
+
+./runtests.sh -t $TEST
--
2.5.5
_______________________________________________
kernel mailing list
kernel(a)lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/kernel@lists.fedoraproject.org