@matheusbsbs / matheus-bagliano / commits / 8024ea60db
Fork 0

bash: support user-supplied completion scripts for aliases

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>

SZEDER Gábor committed Feb 23, 2010 at 22:02 UTC 8024ea60db290dd64db294e1019a0ada64ce1770
1 file changed +7 -4
contrib/completion/git-completion.bash
+7 -4
@@ -2216,11 +2216,14 @@ _git ()
2216 return
2217 fi
2218
2219 - local expansion=$(__git_aliased_command "$command")
2220 - [ "$expansion" ] && command="$expansion"
2221 -
2219 local completion_func="_git_${command//-/_}"
2223 - declare -F $completion_func >/dev/null && $completion_func
2220 + declare -F $completion_func >/dev/null && $completion_func && return
2221 +
2222 + local expansion=$(__git_aliased_command "$command")
2223 + if [ -n "$expansion" ]; then
2224 + completion_func="_git_${expansion//-/_}"
2225 + declare -F $completion_func >/dev/null && $completion_func
2226 + fi
2227 }
2228
2229 _gitk ()