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

[RFC/PATCH 4/5] completion: get rid of compgen

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Nov 17, 2012, 01:38 UTC
Message-ID
<1353116298-11798-5-git-send-email-felipe.contreras@gmail.com>
In-Reply-To
<1353116298-11798-1-git-send-email-felipe.contreras@gmail.com>

The functionality we use is very simple, plus, this fixes a known breakage 'complete tree filename with metacharacters'.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 contrib/completion/git-completion.bash | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index 975ae13..ad3e1fe 100644
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -227,7 +227,11 @@ fi
 
 __gitcompadd ()
 {
-	COMPREPLY=($(compgen -W "$1" -P "$2" -S "$4" -- "$3"))
+	for x in $1; do
+		if [[ "$x" = "$3"* ]]; then
+			COMPREPLY+=("$2$x$4")
+		fi
+	done
 }
 
 # Generates completion reply with compgen, appending a space to possible
-- 
1.8.0
Previous: SZEDER GáborNext: SZEDER Gábor
Message 6 of 15 in “completion: compgen/compadd cleanups”
  1. 0/5 completion: compgen/compadd cleanupsFelipe Contreras, Nov 17, 2012
  2. 1/5 completion: get rid of empty COMPREPLY assignmentsFelipe Contreras, Nov 17, 2012
  3. 2/5 completion: add new __gitcompadd helperFelipe Contreras, Nov 17, 2012
  4. 3/5 completion: trivial test improvementFelipe Contreras, Nov 17, 2012
  5. SZEDER GáborNov 17, 2012
  6. 4/5 completion: get rid of compgenFelipe Contreras, Nov 17, 2012
  7. SZEDER GáborNov 17, 2012
  8. Felipe ContrerasNov 17, 2012
  9. SZEDER GáborNov 17, 2012
  10. Felipe ContrerasNov 17, 2012
  11. Felipe ContrerasNov 18, 2012
  12. 5/5 completion: refactor __gitcomp_1Felipe Contreras, Nov 17, 2012
  13. SZEDER GáborNov 17, 2012
  14. Felipe ContrerasNov 17, 2012
  15. SZEDER GáborNov 17, 2012

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.