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

Re: [PATCH v2 try2 3/4] completion: bash: fix for multiple dash commands

From
SZEDER Gábor <szeder.dev@gmail.com>
Date
Jun 7, 2021, 19:00 UTC
Message-ID
<20210607190046.GB6312@szeder.dev>
In-Reply-To
<20210606184726.1018460-4-felipe.contreras@gmail.com>
On Sun, Jun 06, 2021 at 01:47:25PM -0500, Felipe Contreras wrote:
> Otherwise commands like 'for-each-ref' are not completed.

I think you meant that options of commands like 'for-each-ref' are not completed. The command itself can be completed just fine:

  $ git for<TAB>
  for-each-ref      format-patch      
Show 21 quoted lines
> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
> ---
>  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 05606609f9..1feb2ee108 100644
> --- a/contrib/completion/git-completion.bash
> +++ b/contrib/completion/git-completion.bash
> @@ -421,7 +421,7 @@ __gitcomp_builtin ()
>  	local incl="${2-}"
>  	local excl="${3-}"
>  
> -	local var=__gitcomp_builtin_"${cmd/-/_}"
> +	local var=__gitcomp_builtin_"${cmd//-/_}"
>  	local options
>  	eval "options=\${$var-}"
>  
> -- 
> 2.31.1.2.g0532ba4bf6
> 
Previous: Felipe ContrerasNext: Felipe Contreras
Message 5 of 16 in “completion: bash: a bunch of fixes”
  1. 0/4 completion: bash: a bunch of fixesFelipe Contreras, Jun 6, 2021
  2. 2/4 completion: bash: fix for suboptions with valueFelipe Contreras, Jun 6, 2021
  3. 1/4 completion: bash: fix prefix detection in branch.*Felipe Contreras, Jun 6, 2021
  4. 3/4 completion: bash: fix for multiple dash commandsFelipe Contreras, Jun 6, 2021
  5. SZEDER GáborJun 7, 2021
  6. Felipe ContrerasJun 7, 2021
  7. Junio C HamanoJun 8, 2021
  8. Felipe ContrerasJun 8, 2021
  9. 4/4 completion: bash: add correct suffix in variablesFelipe Contreras, Jun 6, 2021
  10. David AguilarJun 7, 2021
  11. Junio C HamanoJun 7, 2021
  12. 0/4 completion: bash: a bunch of fixesFelipe Contreras, Jun 8, 2021
  13. 2/4 completion: bash: fix for suboptions with valueFelipe Contreras, Jun 8, 2021
  14. 3/4 completion: bash: fix for multiple dash commandsFelipe Contreras, Jun 8, 2021
  15. 1/4 completion: bash: fix prefix detection in branch.*Felipe Contreras, Jun 8, 2021
  16. 4/4 completion: bash: add correct suffix in variablesFelipe Contreras, Jun 8, 2021

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.