Code

grep portability fix: don't use "-e" or "-q"
authorJeff King <peff@peff.net>
Wed, 12 Mar 2008 21:32:17 +0000 (17:32 -0400)
committerJunio C Hamano <gitster@pobox.com>
Thu, 13 Mar 2008 07:57:52 +0000 (00:57 -0700)
commitaadbe44f883859536c5320e0ac1d6ed122c45671
tree5552c7d0999106d0198ed59403b21dcc1c5111e6
parente85fe4d85bc7654af20ccf8054ab6922665405e5
grep portability fix: don't use "-e" or "-q"

System V versions of grep (such as Solaris /usr/bin/grep)
don't understand either of these options. git's usage of
"grep -e pattern" fell into one of two categories:

 1. equivalent to "grep pattern". -e is only useful here if
    the pattern begins with a "-", but all of the patterns
    are hardcoded and do not begin with a dash.

 2. stripping comments and blank lines with

      grep -v -e "^$" -e "^#"

    We can fortunately do this in the affirmative as

      grep '^[^#]'

Uses of "-q" can be replaced with redirection to /dev/null.
In many tests, however, "grep -q" is used as "if this string
is in the expected output, we are OK". In this case, it is
fine to just remove the "-q" entirely; it simply makes the
"verbose" mode of the test slightly more verbose.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-rebase--interactive.sh
git-submodule.sh
t/t0030-stripspace.sh
t/t3404-rebase-interactive.sh
t/t3800-mktag.sh
t/t5400-send-pack.sh
t/t7502-status.sh
t/t7600-merge.sh
t/t9400-git-cvsserver-server.sh