git-sh-setup: remove unused sane_egrep() function

avar · gitster · commit d7927d428cdb · 2021-10-21T16:17:57.000-07:00
The is_zero_oid() function in git-submodule.sh has not been used since e83e333 (submodule: port submodule subcommand 'summary' from shell to C, 2020-08-13), so we can remove it, and the sane_egrep() function, dead is_zero_oid() was the only function which still referenced it. Unlike some other functions in git-sh-setup.sh, this function has not been documented in git-sh-setup(1), so per [1] it should be OK to remove it. I'm still unclear about the future of some of the other functions[2], but any questions in that area should not apply here. 1. https://lore.kernel.org/git/xmqqr1dtgnn8.fsf@gitster.g/ 1. https://lore.kernel.org/git/87tuiwjfvi.fsf@evledraar.gmail.com/ Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
diff --git a/git-sh-setup.sh b/git-sh-setup.sh
@@ -177,10 +177,6 @@ sane_grep () {
 	GREP_OPTIONS= LC_ALL=C grep @@SANE_TEXT_GREP@@ "$@"
 }
 
-sane_egrep () {
-	GREP_OPTIONS= LC_ALL=C egrep @@SANE_TEXT_GREP@@ "$@"
-}
-
 is_bare_repository () {
 	git rev-parse --is-bare-repository
 }

Original file line number	Diff line number	Diff line change
`@@ -177,10 +177,6 @@ sane_grep () {`
`177`	`177`	`GREP_OPTIONS= LC_ALL=C grep @@SANE_TEXT_GREP@@ "$@"`
`178`	`178`	`}`
`179`	`179`
`180`		`-sane_egrep () {`
`181`		`- GREP_OPTIONS= LC_ALL=C egrep @@SANE_TEXT_GREP@@ "$@"`
`182`		`-}`
`183`		`-`
`184`	`180`	`is_bare_repository () {`
`185`	`181`	`git rev-parse --is-bare-repository`
`186`	`182`	`}`