diff options
| author | Shiyang Ruan <ruansy.fnst@fujitsu.com> | 2023-09-08 13:43:35 +0800 |
|---|---|---|
| committer | Zorro Lang <zlang@kernel.org> | 2023-09-20 15:49:54 +0800 |
| commit | f738aca836ad4f6b96673831f28613d8105b5fba (patch) | |
| tree | 32b273101f6c1b3b0a501a97ad86d8eb44f91a0c | |
| parent | 1348ed0e256a21a81cbee6b8c93f10b5273cc118 (diff) | |
| download | xfstests-dev-f738aca836ad4f6b96673831f28613d8105b5fba.tar.gz | |
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 <ruansy.fnst@fujitsu.com>
Reviewed-by: Zorro Lang <zlang@redhat.com>
Reviewed-by: "Darrick J. Wong" <djwong@kernel.org>
Signed-off-by: Zorro Lang <zlang@kernel.org>
| -rwxr-xr-x | tools/mvtest | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tools/mvtest b/tools/mvtest index 99b1541429..09a9982b7f 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" |