]> git.ipfire.org Git - thirdparty/git.git/commitdiff
t0024: avoid losing exit status to pipes
authorGhanshyam Thakkar <shyamthakkar001@gmail.com>
Fri, 19 Jan 2024 03:33:34 +0000 (09:03 +0530)
committerJunio C Hamano <gitster@pobox.com>
Sat, 20 Jan 2024 17:20:39 +0000 (09:20 -0800)
Replace pipe with redirection operator '>' to store the output
to a temporary file after 'git archive' command since the pipe
will swallow the command's exit code and a crash won't
necessarily be noticed.

Also fix an unwanted space after redirection '>' to match the
style described in CodingGuidelines.

Signed-off-by: Ghanshyam Thakkar <shyamthakkar001@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t0024-crlf-archive.sh

index a34de5642073d1323b37b7c82f56edfe798fcfdd..fa4da7c2b3f5a6271db7d40957a3f9e1ba4996dc 100755 (executable)
@@ -9,7 +9,7 @@ test_expect_success setup '
 
        git config core.autocrlf true &&
 
-       printf "CRLF line ending\r\nAnd another\r\n" > sample &&
+       printf "CRLF line ending\r\nAnd another\r\n" >sample &&
        git add sample &&
 
        test_tick &&
@@ -19,8 +19,8 @@ test_expect_success setup '
 
 test_expect_success 'tar archive' '
 
-       git archive --format=tar HEAD |
-       ( mkdir untarred && cd untarred && "$TAR" -xf - ) &&
+       git archive --format=tar HEAD >test.tar &&
+       ( mkdir untarred && cd untarred && "$TAR" -xf ../test.tar ) &&
 
        test_cmp sample untarred/sample