diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2022-03-07 15:49:04 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-03-08 00:27:41 +0300 |
commit | e19b3199506a47098d1ca7b21a5e37cabb70f7f2 (patch) | |
tree | ab9e854f0768a6c24bb4272f5fde8d01061ca729 /t/t4128-apply-root.sh | |
parent | 72aae6d60136da0c727344539003680dc01c9d18 (diff) |
apply tests: don't ignore "git ls-files" exit code, drop sub-shell
Fix code added in 969c877506c (git apply --directory broken for new
files, 2008-10-12) so that it doesn't invoke "git ls-files" on the
left-hand-side of a pipe, instead let's use an intermediate file.
Since we're doing that we can also drop the sub-shell that was here to
group the two.
There are a lot of these sorts of patterns in the test suite, and
there's no particular reason to fix this one other than in a preceding
commit all similar patterns except this one were fixed in
"t/t4128-apply-root.sh", so let's fix this one straggler as well.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t4128-apply-root.sh')
-rwxr-xr-x | t/t4128-apply-root.sh | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/t/t4128-apply-root.sh b/t/t4128-apply-root.sh index ba89a2f2d7..f6db5a79dd 100755 --- a/t/t4128-apply-root.sh +++ b/t/t4128-apply-root.sh @@ -96,7 +96,8 @@ test_expect_success 'apply --directory (delete file)' ' echo content >some/sub/dir/delfile && git add some/sub/dir/delfile && git apply --directory=some/sub/dir/ --index patch && - ! (git ls-files | grep delfile) + git ls-files >out && + ! grep delfile out ' cat > patch << 'EOF' |