[PATCH] Add interactive option in rebase command completion list.

[PATCH] Add interactive option in rebase command completion list.

[Pascal Obry]

Signed-off-by: Pascal Obry <pascal@obry.net>
---
 contrib/completion/git-completion.bash |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index 791e30f..96a712b 100755
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -865,7 +865,7 @@ _git_rebase ()
 		return
 		;;
 	--*)
-		__gitcomp "--onto --merge --strategy"
+		__gitcomp "--onto --merge --strategy --interactive"
 		return
 	esac
 	__gitcomp "$(__git_refs)"
-- 
1.5.5.rc3.7.gba133

Re: [PATCH] Add interactive option in rebase command completion list.

[Junio C Hamano]

Pascal Obry <pascal.obry@gmail.com> writes:

> -		__gitcomp "--onto --merge --strategy"
> +		__gitcomp "--onto --merge --strategy --interactive"

I personally do not think this is worth it where a single "-i" is enough.

Re: [PATCH] Add interactive option in rebase command completion list.

[Elijah Newren]

On Sun, Apr 6, 2008 at 12:19 PM, Junio C Hamano <gitster@pobox.com> wrote:
> Pascal Obry <pascal.obry@gmail.com> writes:
>
>  > -             __gitcomp "--onto --merge --strategy"
>  > +             __gitcomp "--onto --merge --strategy --interactive"
>
>  I personally do not think this is worth it where a single "-i" is enough.

One advantage of bash completion, in my opinion, is being able to be
reminded of option names by tabbing twice to get a list of possible
completions.  For exactly such a case, I'd be in favor of this patch.

Just my $0.02,
Elijah

Re: [PATCH] Add interactive option in rebase command completion list.

[Pascal Obry]

Junio C Hamano a écrit :
> Pascal Obry <pascal.obry@gmail.com> writes:
> 
>> -		__gitcomp "--onto --merge --strategy"
>> +		__gitcomp "--onto --merge --strategy --interactive"
> 
> I personally do not think this is worth it where a single "-i" is enough.

I like unambiguous name (I do prefer --interactive instead of -i). It is 
painful to enter so the bash completion come quite handy here. The patch 
is safe, simple and clean, I see no reason to not include it.

Pascal.

-- 

--|------------------------------------------------------
--| Pascal Obry                           Team-Ada Member
--| 45, rue Gabriel Peri - 78114 Magny Les Hameaux FRANCE
--|------------------------------------------------------
--|              http://www.obry.net
--| "The best way to travel is by means of imagination"
--|
--| gpg --keyserver wwwkeys.pgp.net --recv-key C1082595

Re: [PATCH] Add interactive option in rebase command completion list.

[Shawn O. Pearce]

Pascal Obry <pascal@obry.net> wrote:
> Junio C Hamano a écrit :
> >Pascal Obry <pascal.obry@gmail.com> writes:
> >
> >>-		__gitcomp "--onto --merge --strategy"
> >>+		__gitcomp "--onto --merge --strategy --interactive"
> >
> >I personally do not think this is worth it where a single "-i" is enough.
> 
> I like unambiguous name (I do prefer --interactive instead of -i). It is 
> painful to enter so the bash completion come quite handy here. The patch 
> is safe, simple and clean, I see no reason to not include it.

I agree with Pascal here, include the option.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

-- 
Shawn.

Re: [PATCH] Add interactive option in rebase command completion list.

[Shawn O. Pearce]

"Shawn O. Pearce" <spearce@spearce.org> wrote:
> Pascal Obry <pascal@obry.net> wrote:
> > Junio C Hamano a écrit :
> > >Pascal Obry <pascal.obry@gmail.com> writes:
> > >
> > >>-		__gitcomp "--onto --merge --strategy"
> > >>+		__gitcomp "--onto --merge --strategy --interactive"
> > >
> > >I personally do not think this is worth it where a single "-i" is enough.
> > 
> > I like unambiguous name (I do prefer --interactive instead of -i). It is 
> > painful to enter so the bash completion come quite handy here. The patch 
> > is safe, simple and clean, I see no reason to not include it.
> 
> I agree with Pascal here, include the option.
> 
> Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Gaaaah.

I didn't mean SBO. I mean Acked-by!

Stupid fingers.

-- 
Shawn.