git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 2/4] Teach bash about git-commit's options.

From
Shawn O. Pearce <spearce@spearce.org>
Date
Nov 27, 2006, 20:11 UTC
Message-ID
<20061127201110.GB7308@spearce.org>
In-Reply-To
<139669072795537547ec26b69115509100dcd7ec.1164658223.git.spearce@spearce.org>

Not only can we complete the long options for git-commit, but now we can also complete author strings by looking up all prior author names in all branches and offering them up as possible completions. This is similiar to the support used by the --author= option to git-log.

Unfortunately this changes names which use ' within their string. It seems to be bash's fault here, the ' is being passed out by sed but is getting removed during the completion generation. Fortunately ' appears in only one name in git.git so the damage is likely to be rather low.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
---
 contrib/completion/git-completion.bash |   28 ++++++++++++++++++++++++++++
 1 files changed, 28 insertions(+), 0 deletions(-)
diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index e53f040..d4eb138 100755
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -332,6 +332,32 @@ _git_cherry_pick ()
 	esac
 }
 
+_git_commit ()
+{
+	local cur="${COMP_WORDS[COMP_CWORD]}"
+	case "$cur" in
+	--author=*)
+		local IFS=$'\n'
+		cur="${cur##--author=}"
+		COMPREPLY=($(compgen -P \' -S \' \
+			-W "$(git log --pretty=raw --all \
+			| sed -n -e "/^author /{
+				s/^author //
+				s/>.*/>/
+				p
+			}")" -- "${cur##\'}"))
+		return
+		;;
+	--*)
+		COMPREPLY=($(compgen -W "
+			--all --author=\' --signoff --verify --no-verify
+			--edit --amend --include --only
+			" -- "$cur"))
+		return
+	esac
+	COMPREPLY=()
+}
+
 _git_diff ()
 {
 	__git_complete_file
@@ -720,6 +746,7 @@ _git ()
 	cat-file)    _git_cat_file ;;
 	checkout)    _git_checkout ;;
 	cherry-pick) _git_cherry_pick ;;
+	commit)      _git_commit ;;
 	diff)        _git_diff ;;
 	diff-tree)   _git_diff_tree ;;
 	fetch)       _git_fetch ;;
@@ -754,6 +781,7 @@ complete -o default            -F _git_branch git-branch
 complete -o default -o nospace -F _git_cat_file git-cat-file
 complete -o default            -F _git_checkout git-checkout
 complete -o default            -F _git_cherry_pick git-cherry-pick
+complete -o default            -F _git_commit git-commit
 complete -o default -o nospace -F _git_diff git-diff
 complete -o default            -F _git_diff_tree git-diff-tree
 complete -o default -o nospace -F _git_fetch git-fetch
-- 
1.4.4.1.ge3fb
Next: Shawn O. Pearce
Message 1 of 3 in “Teach bash about git-commit's options.”
  1. 2/4 Teach bash about git-commit's options.Shawn O. Pearce, Nov 27, 2006
  2. 3/4 Cache the list of merge strategies and available commands during load.Shawn O. Pearce, Nov 27, 2006
  3. 4/4 Teach bash about git-am/git-apply and their whitespace options.Shawn O. Pearce, Nov 27, 2006

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.