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

[PATCH] contrib/completion: suppress stderror in bash completion of git remotes

From
Matt Korostoff <mkorostoff@gmail.com>
Date
Feb 9, 2015, 20:58 UTC
Message-ID
<1423515508-40109-1-git-send-email-MKorostoff@gmail.com>

In some system configurations there is a bug with the __git_remotes function. Specifically, there is a problem with line 415, `test -d "$d/remotes" && ls -1 "$d/remotes"`. While `test -d` is meant to prevent listing the remotes directory if it does not exist, in some system, `ls` will run regardless.

This results in an error in which typing `git push or` + `tab`
prints out `ls: .git/remotes: No such file or directory`.
This can be fixed by simply directing stderror of this line
to /dev/null.
---
 contrib/completion/git-completion.bash |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index 2fece98..72251cc 100644
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -412,7 +412,7 @@ __git_refs_remotes ()
 __git_remotes ()
 {
 	local i IFS=$'\n' d="$(__gitdir)"
-	test -d "$d/remotes" && ls -1 "$d/remotes"
+	test -d "$d/remotes" && ls -1 "$d/remotes" 2>/dev/null
 	for i in $(git --git-dir="$d" config --get-regexp 'remote\..*\.url' 2>/dev/null); do
 		i="${i#remote.}"
 		echo "${i/.url*/}"
-- 
1.7.10.2 (Apple Git-33)
Next: Matt Korostoff
Message 1 of 11 in “contrib/completion: suppress stderror in bash completion of git remotes”
  1. contrib/completion: suppress stderror in bash completion of git remotesMatt Korostoff, Feb 9, 2015
  2. Matt KorostoffFeb 9, 2015
  3. Junio C HamanoFeb 9, 2015
  4. Matt KorostoffFeb 10, 2015
  5. SZEDER GáborFeb 10, 2015
  6. contrib/completion: deprecate __git_remotes in bash completionMatt Korostoff, Feb 10, 2015
  7. Junio C HamanoFeb 10, 2015
  8. SZEDER GáborFeb 10, 2015
  9. SZEDER GáborMar 4, 2015
  10. 1/2 completion: add a test for __git_remotes() helper functionSZEDER Gábor, Mar 4, 2015
  11. 2/2 completion: simplify __git_remotes()SZEDER Gábor, Mar 4, 2015

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.