On Fri, Jun 28, 2013 at 07:48:07PM +0530, Ramkumar Ramachandra wrote: > Signed-off-by: Ramkumar Ramachandra <artagnon@xxxxxxxxx> > --- > contrib/completion/git-completion.bash | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash > index 278018f..f2959a7 100644 > --- a/contrib/completion/git-completion.bash > +++ b/contrib/completion/git-completion.bash > @@ -2247,6 +2247,20 @@ _git_reset () > __gitcomp_nl "$(__git_refs)" > } > > +_git_rev_parse () > +{ > + case "$cur" in > + --*) > + __gitcomp " > + --short --show-toplevel --is-inside-work-tree > + --symbolic-full-name --verify > + " In the completion script we support porcelain commands. I'm not sure about 'git rev-parse', but I think it's more plumbing than porcelain. However, I think the same about 'git ls-tree' and 'git reflog', too, yet we have support for them in the completion script. Either way, why these five options? 'git rev-parse' has a lot more options than that. > + return > + ;; > + esac > + __gitcomp_nl "$(__git_refs)" > +} > + > _git_revert () > { > case "$cur" in > -- > 1.8.3.1.585.g9832cb9 > -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html