# [PATCH 4/9] add missing long option to 'git grep' bash competion

3 messages from 2009-08-08 to 2009-08-08. Participants: Emmanuel Trillaud, Junio C Hamano.
Thread: https://gitlist.dev/t/20475

## Emmanuel Trillaud, 2009-08-08 08:56

Subject: [PATCH 4/9] add missing long option to 'git grep' bash competion
Message-ID: <9f50533b0908080156h23415740lb94cbd9f2c6a165d@mail.gmail.com>
URL: https://gitlist.dev/e/9f50533b0908080156h23415740lb94cbd9f2c6a165d%40mail.gmail.com

```
according to the 'git-grep' man page, the following long options are missing
from the 'git grep' bash completion :
--null
--color
--no-color

Signed-off-by: Emmanuel Trillaud <etrillaud@gmail.com>
---
 contrib/completion/git-completion.bash |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/contrib/completion/git-completion.bash
b/contrib/completion/git-completion.bash
index f56733b..a4e78d5 100755
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -1057,7 +1057,9 @@ _git_grep ()
 			--extended-regexp --basic-regexp --fixed-strings
 			--files-with-matches --name-only
 			--files-without-match
+			--null
 			--count
+			--color --no-color
 			--and --or --not --all-match
 			"
 		return
-- 
1.6.4

```

## Junio C Hamano, 2009-08-08 19:48

Subject: Re: [PATCH 4/9] add missing long option to 'git grep' bash competion
Message-ID: <7vskg2dr3o.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vskg2dr3o.fsf%40alter.siamese.dyndns.org
In-Reply-To: <9f50533b0908080156h23415740lb94cbd9f2c6a165d@mail.gmail.com>

```
Emmanuel Trillaud <etrillaud@gmail.com> writes:

> according to the 'git-grep' man page, the following long options are missing
> from the 'git grep' bash completion :
> --null
> --color
> --no-color

I do not think completion's purpose should be to offer everything
available under the sun, so "according to manpage these are missing" can
never be a good justification for patches in this series.

Does it even make sense to complete --null in an interactive invocation?

```

## Emmanuel Trillaud, 2009-08-08 20:07

Subject: Re: [PATCH 4/9] add missing long option to 'git grep' bash competion
Message-ID: <20090808220739.663bb1f7@eleanor>
URL: https://gitlist.dev/e/20090808220739.663bb1f7%40eleanor
In-Reply-To: <7vskg2dr3o.fsf@alter.siamese.dyndns.org>

```
> > according to the 'git-grep' man page, the following long options are
> > missing from the 'git grep' bash completion :
> > --null
> > --color
> > --no-color
> 
> I do not think completion's purpose should be to offer everything
> available under the sun, so "according to manpage these are missing" can
> never be a good justification for patches in this series.
When I saw that completion for the long options of 'git checkout' was
mising, I just start looking around the docs to see if there wasn't other
git commands with some missing completions. I see now it was a wrong way
to update the git bash completion. I will soon come back with a new set
of (more reasonnable) patches.

Emmanuel 

```