diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2021-10-21 19:58:00 (GMT) |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-10-21 23:17:57 (GMT) |
commit | ebeb39faad6e3a67c31884c3dc6b76ce58b3f15b (patch) | |
tree | 1139ff6cd0221247f64c16a9f4d4180cdcfdfe26 /git-filter-branch.sh | |
parent | d7927d428cdb6ed0d709fea2e5353d56833ae02e (diff) | |
download | git-ebeb39faad6e3a67c31884c3dc6b76ce58b3f15b.zip git-ebeb39faad6e3a67c31884c3dc6b76ce58b3f15b.tar.gz git-ebeb39faad6e3a67c31884c3dc6b76ce58b3f15b.tar.bz2 |
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>
Diffstat (limited to 'git-filter-branch.sh')
-rwxr-xr-x | git-filter-branch.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/git-filter-branch.sh b/git-filter-branch.sh index cb89372..3a51d45 100755 --- a/git-filter-branch.sh +++ b/git-filter-branch.sh @@ -579,7 +579,7 @@ if [ "$filter_tag_name" ]; then git hash-object -t tag -w --stdin) || die "Could not create new tag object for $ref" if git cat-file tag "$ref" | \ - sane_grep '^-----BEGIN PGP SIGNATURE-----' >/dev/null 2>&1 + grep '^-----BEGIN PGP SIGNATURE-----' >/dev/null 2>&1 then warn "gpg signature stripped from tag object $sha1t" fi |