On Fri, Sep 08, 2023 at 01:43:35PM +0800, Shiyang Ruan wrote:
> 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@xxxxxxxxxxx>
> Reviewed-by: Zorro Lang <zlang@xxxxxxxxxx>
> ---
> tools/mvtest | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/tools/mvtest b/tools/mvtest
> index 99b154142..bd4006eb2 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 755 "tests/${dest}"
No objections to forcing 0755, though "a+x" here would have gotten the
job done as well.
Reviewed-by: Darrick J. Wong <djwong@xxxxxxxxxx>
--D
> 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.42.0
>
