On Thu, Apr 28, 2011 at 7:01 PM, SZEDER GÃbor <szeder@xxxxxxxxxx> wrote:
> The "_get_comp_words_by_ref -n := words" command from the
> bash_completion library reassembles a modified version of COMP_WORDS
> with ':' and '=' no longer treated as word separators and stores it in
> the ${words[@]} array. ÂGit's programmable tab completion script uses
> this to abstract away the difference between bash v3's and bash v4's
> definitions of COMP_WORDS (bash v3 used shell words, while bash v4
> breaks at separator characters); see v1.7.4-rc0~11^2~2 (bash: get
> --pretty=m<tab> completion to work with bash v4, 2010-12-02).
>
> zsh has (or rather its completion functions have) another idea about
> what ${words[@]} should contain: the array is prepopulated with the
> words from the command it is completing. ÂFor reasons that are not
> well understood, when git-completion.bash reserves its own "words"
> variable with "local words", the variable becomes empty and cannot be
> changed from then on. ÂSo the completion script neglects the arguments
> it has seen, and words complete like git subcommand names. ÂFor
> example, typing "git log origi<TAB>" gives no completions because
> there are no "git origi..." commands.
>
> However, when this words variable is not declared as local but is just
> populated by _get_comp_words_by_ref() and then read in various
> completion functions, then zsh seems to be happy about it and our
> completion script works as expected.
>
> So, to get our completion script working again under zsh and to
> prevent the words variable from leaking into the shell environment
> under bash, we will only declare words as local when using bash.
>
> Reported-by: Stefan Haller <lists@xxxxxxxxxxxxxxxx>
> Suggested-by: Felipe Contreras <felipe.contreras@xxxxxxxxx>
> Explained-by: Jonathan Nieder <jrnieder@xxxxxxxxx>
> Signed-off-by: SZEDER GÃbor <szeder@xxxxxxxxxx>
> ---
> Âcontrib/completion/git-completion.bash | Â Â8 ++++++--
> Â1 files changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
> index 862b840..6869765 100755
> --- a/contrib/completion/git-completion.bash
> +++ b/contrib/completion/git-completion.bash
> @@ -2608,9 +2608,11 @@ _git ()
> Â Â Â Âif [[ -n ${ZSH_VERSION-} ]]; then
> Â Â Â Â Â Â Â Âemulate -L bash
> Â Â Â Â Â Â Â Âsetopt KSH_TYPESET
> + Â Â Â else
> + Â Â Â Â Â Â Â local words
> Â Â Â Âfi
>
> - Â Â Â local cur words cword prev
> + Â Â Â local cur cword prev
> Â Â Â Â_get_comp_words_by_ref -n =: cur words cword prev
> Â Â Â Âwhile [ $c -lt $cword ]; do
> Â Â Â Â Â Â Â Âi="${words[c]}"
> @@ -2659,9 +2661,11 @@ _gitk ()
> Â Â Â Âif [[ -n ${ZSH_VERSION-} ]]; then
> Â Â Â Â Â Â Â Âemulate -L bash
> Â Â Â Â Â Â Â Âsetopt KSH_TYPESET
> + Â Â Â else
> + Â Â Â Â Â Â Â local words
> Â Â Â Âfi
>
> - Â Â Â local cur words cword prev
> + Â Â Â local cur cword prev
> Â Â Â Â_get_comp_words_by_ref -n =: cur words cword prev
>
> Â Â Â Â__git_has_doubledash && return
> --
> 1.7.5.86.g799a6
Here's another option:
>From 603e4db259283a4eb6bac2315a630480e3238f50 Mon Sep 17 00:00:00 2001
From: Felipe Contreras <felipe.contreras@xxxxxxxxx>
Date: Tue, 3 May 2011 20:45:26 +0300
Subject: [PATCH] git-completion: fix zsh support
It turns out 'words' is a special variable used by zsh completion.
There's probably a bug in zsh's bashcompinit:
http://article.gmane.org/gmane.comp.shells.zsh.devel/22546
But in the meantime we can workaround it this way.
Signed-off-by: Felipe Contreras <felipe.contreras@xxxxxxxxx>
---
contrib/completion/git-completion.bash | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/contrib/completion/git-completion.bash
b/contrib/completion/git-completion.bash
index 00691fc..d32b1b8 100755
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -2608,6 +2608,9 @@ _git ()
if [[ -n ${ZSH_VERSION-} ]]; then
emulate -L bash
setopt KSH_TYPESET
+
+ # 'words' has special meaning in zsh; override that
+ typeset -h words
fi
local cur words cword prev
--
1.7.5
--
Felipe Contreras
--
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