Skip to content

Commit fcd9d70

Browse files
peffgitster
authored andcommitted
t5561: drop curl stderr redirects
For a normal test run, stderr is already redirected to /dev/null by the test suite. When used with "-v", suppressing stderr is actively harmful, as it may hide the reason for curl failing. Reported-by: Jens Krüger <[email protected]> Signed-off-by: Jeff King <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent d32eb83 commit fcd9d70

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

t/t5561-http-backend.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ test_description='test git-http-backend'
66
start_httpd
77

88
GET() {
9-
curl --include "$HTTPD_URL/$SMART/repo.git/$1" >out 2>/dev/null &&
9+
curl --include "$HTTPD_URL/$SMART/repo.git/$1" >out &&
1010
tr '\015' Q <out |
1111
sed '
1212
s/Q$//
@@ -19,7 +19,7 @@ GET() {
1919
POST() {
2020
curl --include --data "$2" \
2121
--header "Content-Type: application/x-$1-request" \
22-
"$HTTPD_URL/smart/repo.git/$1" >out 2>/dev/null &&
22+
"$HTTPD_URL/smart/repo.git/$1" >out &&
2323
tr '\015' Q <out |
2424
sed '
2525
s/Q$//

0 commit comments

Comments
 (0)