| Message ID | 20230908054335.13625-1-ruansy.fnst@fujitsu.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v3] tools/mvtest: ensure testcase is executable (755) | expand |
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@fujitsu.com> > Reviewed-by: Zorro Lang <zlang@redhat.com> > --- > 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@kernel.org> --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 >
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}" 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"
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> --- tools/mvtest | 2 ++ 1 file changed, 2 insertions(+)