]> git.ipfire.org Git - thirdparty/git.git/commit
git-sh-setup: remove "sane_grep", it's not needed anymore
authorÆvar Arnfjörð Bjarmason <avarab@gmail.com>
Thu, 21 Oct 2021 19:58:00 +0000 (21:58 +0200)
committerJunio C Hamano <gitster@pobox.com>
Thu, 21 Oct 2021 23:17:57 +0000 (16:17 -0700)
commitebeb39faad6e3a67c31884c3dc6b76ce58b3f15b
tree1139ff6cd0221247f64c16a9f4d4180cdcfdfe26
parentd7927d428cdb6ed0d709fea2e5353d56833ae02e
git-sh-setup: remove "sane_grep", it's not needed anymore

Remove the sane_grep() shell function in git-sh-setup. The two reasons
for why it existed don't apply anymore:

1. It was added due to GNU grep supporting GREP_OPTIONS. See
   e1622bfcbad (Protect scripted Porcelains from GREP_OPTIONS insanity,
   2009-11-23).

   Newer versions of GNU grep ignore that, but even on older versions
   its existence won't matter, none of these sane_grep() uses care
   about grep's output, they're merely using it to check if a string
   exists in a file or stream. We also don't care about the "LC_ALL=C"
   that "sane_grep" was using, these greps for fixed or ASCII strings
   will behave the same under any locale.

2. The SANE_TEXT_GREP added in 71b401032b9 (sane_grep: pass "-a" if
   grep accepts it, 2016-03-08) isn't needed either, none of these grep
   uses deal with binary data.

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Makefile
config.mak.uname
configure.ac
contrib/buildsystems/CMakeLists.txt
git-filter-branch.sh
git-instaweb.sh
git-sh-setup.sh