Re: bash completion only provides revs, not paths
Uwe Kleine-Knig <Uwe.Kleine-Koenig@digi.com> wrote:
Show 6 quoted lines
> currently autocompletion in
>
> ukleinek@zentaur:~/gsrc/linux-2.6 git log ar<tab>
>
> only yields "armltd/" (i.e. the prefix of some remote tracking branches)
> but most of the time I want it to complete to "arch/".
Right. It favors refs over paths. Why? Because I complete refs
more than I complete paths.
Show 5 quoted lines
> I don't understand that autocompletion stuff, but probably to fix that
> several __git_complete_revlist should be replaced by a
> __git_complete_revlist_or_path. The harder part is to implement the
> latter function. An optimal implementation would only give back paths
> if there is a "--" somewhere before the cursor.
Its ugly, but if we only do path name completion after a -- then we can just return "" as the options to bash, and it will go off and do the path name completion itself. If we do path names and and refs at the same time, that's a disaster waiting to happen. We would have to do the path name completion ourselves.
How is this?
--8<-- bash completion: Completely only path names after -- in options
If the user has inserted a -- before the current argument we are being asked to complete they are most likely trying to force Git into treating this argument as a path name, and not as a ref. We should allow bash to complete it as a path from the local working directory.
Suggested-by: Uwe Kleine-Knig <Uwe.Kleine-Koenig@digi.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
---
contrib/completion/git-completion.bash | 15 ++++++++++++++-
1 files changed, 14 insertions(+), 1 deletions(-)
diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index 0f5d30c..8d8405e 100755
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -308,7 +308,20 @@ __git_complete_file ()
__git_complete_revlist ()
{
- local pfx cur="${COMP_WORDS[COMP_CWORD]}"
+ local c=1 pfx cur
+
+ while [ $c -lt $COMP_CWORD ]; do
+ cur="${COMP_WORDS[c]}"
+ case "$cur" in
+ --)
+ COMPREPLY=()
+ return
+ ;;
+ esac
+ c=$((++c))
+ done
+
+ cur="${COMP_WORDS[COMP_CWORD]}"
case "$cur" in
*...*)
pfx="${cur%...*}..."
--
1.5.5.1.141.gd55b
--
Shawn.
Uwe Kleine-König