threads / patch / 32162

patchFix bash completion when `egrep` is aliased to `egrep --color=always`

Subject: [PATCH] Fix bash completion when `egrep` is aliased to `egrep --color=always`

## tl;dr

4 messages between Nov 22, 2012 and Nov 26, 2012. Diffs are folded; open one to read it.

replies: 3people: 3as markdown or json

Adam Tkac· Nov 22, 2012, 15:41 UTC · lore
Hello all,
attached patch fixes bash completion when `egrep` is aliased to `egrep --color=always`.
Comments are welcomed.
Regards, Adam
-- 
Adam Tkac, Red Hat, Inc.


>From 2b62bd71af1158129492f74f0b77c9840a49507f Mon Sep 17 00:00:00 2001
From: Adam Tkac <atkac@redhat.com>
Date: Thu, 22 Nov 2012 16:34:58 +0100
Subject: [PATCH] Fix bash completion when `egrep` is aliased to `egrep
 --color=always`

Originally reported as https://bugzilla.redhat.com/show_bug.cgi?id=863780

Signed-off-by: Adam Tkac <atkac@redhat.com>
Signed-off-by: Holger Arnold <holgerar@gmail.com>
---
 contrib/completion/git-completion.bash | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index bc0657a..47613f9 100644
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -565,7 +565,7 @@ __git_complete_strategy ()
 __git_list_all_commands ()
 {
 	local i IFS=" "$'\n'
-	for i in $(git help -a|egrep '^  [a-zA-Z0-9]')
+	for i in $(git help -a|egrep --color=never '^  [a-zA-Z0-9]')
 	do
 		case $i in
 		*--*)             : helper pattern;;
-- 
1.8.0
Marc Khouzam· Nov 22, 2012, 19:55 UTC · re: Adam Tkac · lore

Re: [PATCH] Fix bash completion when `egrep` is aliased to `egrep --color=always`

On Thu, Nov 22, 2012 at 10:41 AM, Adam Tkac <atkac@redhat.com> wrote:
> Hello all,
>
> attached patch fixes bash completion when `egrep` is aliased to `egrep --color=always`.

To avoid any aliases, it may be better to use \egrep

This could be worthwhile for all utilities used by the script.
Just a thought.
Marc
Show 7 quoted lines
>
> Comments are welcomed.
>
> Regards, Adam
>
> --
> Adam Tkac, Red Hat, Inc.
Adam Tkac· Nov 26, 2012, 11:23 UTC · re: Marc Khouzam · lore

Re: [PATCH] Fix bash completion when `egrep` is aliased to `egrep --color=always`

On Thu, Nov 22, 2012 at 02:55:21PM -0500, Marc Khouzam wrote:
Show 7 quoted lines
> On Thu, Nov 22, 2012 at 10:41 AM, Adam Tkac <atkac@redhat.com> wrote:
> > Hello all,
> >
> > attached patch fixes bash completion when `egrep` is aliased to `egrep --color=always`.
> 
> To avoid any aliases, it may be better to use
> \egrep
Good idea, thanks. Improved patch is attached.
Regards, Adam
Show 15 quoted lines
> 
> This could be worthwhile for all utilities used by the script.
> 
> Just a thought.
> 
> Marc
> 
> 
> >
> > Comments are welcomed.
> >
> > Regards, Adam
> >
> > --
> > Adam Tkac, Red Hat, Inc.
-- 
Adam Tkac, Red Hat, Inc.


>From 255192296cd175fddcac2647447a66a0ca55b855 Mon Sep 17 00:00:00 2001
From: Adam Tkac <atkac@redhat.com>
Date: Thu, 22 Nov 2012 16:34:58 +0100
Subject: [PATCH] If `egrep` is aliased, temporary disable it in
 bash.completion

Originally reported as https://bugzilla.redhat.com/show_bug.cgi?id=863780

Signed-off-by: Adam Tkac <atkac@redhat.com>
Signed-off-by: Holger Arnold <holgerar@gmail.com>
---
 contrib/completion/git-completion.bash | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index 0960acc..79073c2 100644
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -565,7 +565,7 @@ __git_complete_strategy ()
 __git_list_all_commands ()
 {
 	local i IFS=" "$'\n'
-	for i in $(git help -a|egrep '^  [a-zA-Z0-9]')
+	for i in $(git help -a| \egrep '^  [a-zA-Z0-9]')
 	do
 		case $i in
 		*--*)             : helper pattern;;
-- 
1.8.0
Frans Klaver· Nov 26, 2012, 11:30 UTC · re: Adam Tkac · lore

Re: [PATCH] Fix bash completion when `egrep` is aliased to `egrep --color=always`

On Mon, Nov 26, 2012 at 12:23 PM, Adam Tkac <atkac@redhat.com> wrote:
> Good idea, thanks. Improved patch is attached.

It is custom on this list to mail the patches, rather than attaching them, so people can review your changes in-line. You can read more about it in in git/Documentation/SubmittingPatches.

Cheers, Frans

← back to recent threads