From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 11 Jul 2016 17:44:13 +0000 (-0700)
Subject: Merge branch 'lc/shell-default-value-noexpand' into maint
X-Git-Tag: v2.9.1~10
X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/5220b7589b09d62e7166799baa98dcd6a1c02804?hp=-c

Merge branch 'lc/shell-default-value-noexpand' into maint

Fix unnecessarily waste in the idiomatic use of ': ${VAR=default}'
to set the default value, without enclosing it in double quotes.

* lc/shell-default-value-noexpand:
sh-setup: enclose setting of ${VAR=default} in double-quotes
---

5220b7589b09d62e7166799baa98dcd6a1c02804
diff --combined git-sh-setup.sh
index c48139a494,c918ed779b..85db5f1c63
--- a/git-sh-setup.sh
+++ b/git-sh-setup.sh
@@@ -160,19 -160,19 +160,19 @@@ git_pager() 
  	else
  		GIT_PAGER=cat
  	fi
- 	: ${LESS=-FRX}
- 	: ${LV=-c}
+ 	: "${LESS=-FRX}"
+ 	: "${LV=-c}"
  	export LESS LV
  
  	eval "$GIT_PAGER" '"$@"'
  }
  
  sane_grep () {
 -	GREP_OPTIONS= LC_ALL=C grep "$@"
 +	GREP_OPTIONS= LC_ALL=C grep @@SANE_TEXT_GREP@@ "$@"
  }
  
  sane_egrep () {
 -	GREP_OPTIONS= LC_ALL=C egrep "$@"
 +	GREP_OPTIONS= LC_ALL=C egrep @@SANE_TEXT_GREP@@ "$@"
  }
  
  is_bare_repository () {
@@@ -344,7 -344,7 +344,7 @@@ git_dir_init () 
  		echo >&2 "Unable to determine absolute path of git directory"
  		exit 1
  	}
- 	: ${GIT_OBJECT_DIRECTORY="$(git rev-parse --git-path objects)"}
+ 	: "${GIT_OBJECT_DIRECTORY="$(git rev-parse --git-path objects)"}"
  }
  
  if test -z "$NONGIT_OK"