From f738aca836ad4f6b96673831f28613d8105b5fba Mon Sep 17 00:00:00 2001 From: Shiyang Ruan Date: Fri, 8 Sep 2023 13:43:35 +0800 Subject: [PATCH] tools/mvtest: ensure testcase is executable (755) Some test cases lack executable permission ('x'). Before running each test case, `./check` checks and grants them 'x' permission. However, this always leads to a dirty git repo. And the absence of 'x' permission in test cases is often overlooked during reviews. Since maintainers use mvtest to assign new case, add this change for convenience of maintainers. Signed-off-by: Shiyang Ruan Reviewed-by: Zorro Lang Reviewed-by: "Darrick J. Wong" Signed-off-by: Zorro Lang --- tools/mvtest | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/mvtest b/tools/mvtest index 99b15414..09a9982b 100755 --- a/tools/mvtest +++ b/tools/mvtest @@ -34,6 +34,8 @@ did="$(basename "${dest}")" git mv "tests/${src}" "tests/${dest}" git mv "tests/${src}.out" "tests/${dest}.out" +# make sure testcase is executable +chmod a+x "tests/${dest}" sed -e "s/^# FS[[:space:]]*QA.*Test.*[0-9]\+$/# FS QA Test No. ${did}/g" -i "tests/${dest}" sed -e "s/^QA output created by ${sid}$/QA output created by ${did}/g" -i "tests/${dest}.out" sed -e "s/test-${sid}/test-${did}/g" -i "tests/${dest}.out" -- 2.39.5