pull.sh: quote $upload_pack when passing it to git-fetch

moy · gitster · commit 22d6857d46de · 2015-07-30T14:14:32.000-07:00
The previous code broke for example

  git pull --upload-pack 'echo --foo'

Reported-by: Joey Hess &lt;id@joeyh.name&gt;
Fix-suggested-by: Junio C Hamano &lt;gitster@pobox.com&gt;
Signed-off-by: Matthieu Moy &lt;Matthieu.Moy@imag.fr&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/git-pull.sh b/git-pull.sh
@@ -295,7 +295,7 @@ test true = "$rebase" && {
 }
 orig_head=$(git rev-parse -q --verify HEAD)
 git fetch $verbosity $progress $dry_run $recurse_submodules $all $append \
-$upload_pack $force $tags $prune $keep $depth $unshallow $update_shallow \
+${upload_pack+"$upload_pack"} $force $tags $prune $keep $depth $unshallow $update_shallow \
 $refmap --update-head-ok "$@" || exit 1
 test -z "$dry_run" || exit 0
 

Original file line number	Diff line number	Diff line change
`@@ -295,7 +295,7 @@ test true = "$rebase" && {`
`295`	`295`	`}`
`296`	`296`	`orig_head=$(git rev-parse -q --verify HEAD)`
`297`	`297`	`git fetch $verbosity $progress $dry_run $recurse_submodules $all $append \`
`298`		`-$upload_pack $force $tags $prune $keep $depth $unshallow $update_shallow \`
	`298`	`+${upload_pack+"$upload_pack"} $force $tags $prune $keep $depth $unshallow $update_shallow \`
`299`	`299`	`$refmap --update-head-ok "$@" \|\| exit 1`
`300`	`300`	`test -z "$dry_run" \|\| exit 0`
`301`	`301`