[PATCH] bash: don't offer remote transport helpers as subcommands

[PATCH] bash: don't offer remote transport helpers as subcommands

[SZEDER Gábor]

Since commits a2d725b7 (Use an external program to implement fetching
with curl, 2009-08-05) and c9e388bb (Make the
"traditionally-supported" URLs a special case, 2009-09-03) remote
transport helpers like 'remote-ftp' and 'remote-curl' are offered by the
completion script as available subcommands.  Not good, since they are
helpers, therefore should not be offered, so filter them out.

Signed-off-by: SZEDER Gábor <szeder@ira.uka.de>
---

Maybe maint-worthy?  1.6.5 was the first release with this bug, but
nobody complained since then.

 contrib/completion/git-completion.bash |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index 9651720..7def62c 100755
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -568,6 +568,7 @@ __git_list_porcelain_commands ()
 		read-tree)        : plumbing;;
 		receive-pack)     : plumbing;;
 		reflog)           : plumbing;;
+		remote-*)         : transport;;
 		repo-config)      : deprecated;;
 		rerere)           : plumbing;;
 		rev-list)         : plumbing;;
-- 
1.6.6.1.361.gc5121

Re: [PATCH] bash: don't offer remote transport helpers as subcommands

[Shawn O. Pearce]

SZEDER G??bor <szeder@ira.uka.de> wrote:
> Since commits a2d725b7 (Use an external program to implement fetching
> with curl, 2009-08-05) and c9e388bb (Make the
> "traditionally-supported" URLs a special case, 2009-09-03) remote
> transport helpers like 'remote-ftp' and 'remote-curl' are offered by the
> completion script as available subcommands.  Not good, since they are
> helpers, therefore should not be offered, so filter them out.
> 
> Signed-off-by: SZEDER G??bor <szeder@ira.uka.de>
> ---
> 
> Maybe maint-worthy?  1.6.5 was the first release with this bug, but
> nobody complained since then.

Yup, should be in maint.

Acked-by: Shawn O. Pearce <spearce@spearce.org>
 
>  contrib/completion/git-completion.bash |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
> index 9651720..7def62c 100755
> --- a/contrib/completion/git-completion.bash
> +++ b/contrib/completion/git-completion.bash
> @@ -568,6 +568,7 @@ __git_list_porcelain_commands ()
>  		read-tree)        : plumbing;;
>  		receive-pack)     : plumbing;;
>  		reflog)           : plumbing;;
> +		remote-*)         : transport;;
>  		repo-config)      : deprecated;;
>  		rerere)           : plumbing;;
>  		rev-list)         : plumbing;;
> -- 
> 1.6.6.1.361.gc5121

-- 
Shawn.