Revert "git_connect: prefer Git's builtins over dashed form"

It would appear that this change (which was intended to fix tests
interacting with local repositories when `git-upload-pack` was not in the
`PATH`) regresses on SSH access.

This reverts commit 40023e5 and fixes
#1258.

Signed-off-by: Johannes Schindelin <[email protected]>

Author: dscho

connect.c

@@ -825,9 +825,6 @@ struct child_process *git_connect(int fd[2], const char *url,
 		child_process_init(conn);
 
 		strbuf_addstr(&cmd, prog);
-		/* Prefer the builtin */
-		if (starts_with(prog, "git-"))
-			cmd.buf[3] = ' ';
 		strbuf_addch(&cmd, ' ');
 		sq_quote_buf(&cmd, path);
 

t/t5601-clone.sh

@@ -332,13 +332,13 @@ expect_ssh () {
 		1)
 			;;
 		2)
-			echo "ssh: $1 git upload-pack '$2'"
+			echo "ssh: $1 git-upload-pack '$2'"
 			;;
 		3)
-			echo "ssh: $1 $2 git upload-pack '$3'"
+			echo "ssh: $1 $2 git-upload-pack '$3'"
 			;;
 		*)
-			echo "ssh: $1 $2 git upload-pack '$3' $4"
+			echo "ssh: $1 $2 git-upload-pack '$3' $4"
 		esac
 	} >"$TRASH_DIRECTORY/ssh-expect" &&
 	(cd "$TRASH_DIRECTORY" && test_cmp ssh-expect ssh-output)

t/t5602-clone-remote-exec.sh

@@ -13,7 +13,7 @@ test_expect_success setup '
 
 test_expect_success 'clone calls git upload-pack unqualified with no -u option' '
 	test_must_fail env GIT_SSH=./not_ssh git clone localhost:/path/to/repo junk &&
-	echo "localhost git upload-pack '\''/path/to/repo'\''" >expected &&
+	echo "localhost git-upload-pack '\''/path/to/repo'\''" >expected &&
 	test_cmp expected not_ssh_output
 '