bash: support user-supplied completion scripts for aliases
authorSZEDER Gábor <szeder@ira.uka.de>
Tue, 23 Feb 2010 21:02:59 +0000 (22:02 +0100)
committerJunio C Hamano <gitster@pobox.com>
Wed, 24 Feb 2010 16:32:26 +0000 (08:32 -0800)
Shell command aliases can get rather complex, and the completion
script can not always determine correctly the git command invoked by
such an alias.  For such cases users might want to provide custom
completion scripts the same way like for their custom commands made
possible by the previous patch.

The current completion script does not allow this, because if it
encounters an alias, then it will unconditionally perform completion
for the aliased git command (in case it can determine the aliased git
command, of course).  With this patch the completion script will first
search for a completion function for the command given on the command
line, be it a git command, a custom git command of the user, or an
alias, and invoke that function to perform the completion.  This has
no effect on git commands, because they can not be aliased anyway.  If
it is an alias and there is a completion function for that alias (e.g.
_git_foo() for the alias 'foo'), then it will be invoked to perform
completion, allowing users to provide custom completion functions for
aliases.  If such a completion function can not be found, only then
will the completion script check whether the command given on the
command line is an alias or not, and proceed as usual (i.e. find out
the aliased git command and provide completion for it).

Signed-off-by: SZEDER Gábor <szeder@ira.uka.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
contrib/completion/git-completion.bash

index 2ac356705bd6f94de84d9600b0e24a1ad6c8a3bb..8593fd707b5e80b8d8833454744dbe261fa5e9e5 100755 (executable)
@@ -2216,11 +2216,14 @@ _git ()
                return
        fi
 
-       local expansion=$(__git_aliased_command "$command")
-       [ "$expansion" ] && command="$expansion"
-
        local completion_func="_git_${command//-/_}"
-       declare -F $completion_func >/dev/null && $completion_func
+       declare -F $completion_func >/dev/null && $completion_func && return
+
+       local expansion=$(__git_aliased_command "$command")
+       if [ -n "$expansion" ]; then
+               completion_func="_git_${expansion//-/_}"
+               declare -F $completion_func >/dev/null && $completion_func
+       fi
 }
 
 _gitk ()