The POSIX man page of printf(1) mentions - > If the format operand contains no conversion specifications and > argument operands are present, the results are unspecified. In practice, this means some printf implementations throw an error when provided with extra operands, thereby causing the test to fail erroneously. This commit fixes that issue. Signed-off-by: Subhaditya Nath <sn03.general@xxxxxxxxx> --- t/t7422-submodule-output.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/t7422-submodule-output.sh b/t/t7422-submodule-output.sh index 023a5cbdc4..1f291a1d49 100755 --- a/t/t7422-submodule-output.sh +++ b/t/t7422-submodule-output.sh @@ -180,7 +180,7 @@ test_expect_success !MINGW 'git submodule status --recursive propagates SIGPIPE' COMMIT=$(git rev-parse HEAD) && for i in $(test_seq 2000) do - printf "[submodule \"sm-$i\"]\npath = recursive-submodule-path-$i\n" "$i" || + printf "[submodule \"sm-$i\"]\npath = recursive-submodule-path-$i\n" || return 1 done >gitmodules && BLOB=$(git hash-object -w --stdin <gitmodules) && -- 2.48.1
