Re: [PATCH 1/2] remote: Remove -v/--verbose option from git remote show synopsis

From: Junio C Hamano <gitster@pobox.com>
Date: Jan 8, 2015, 19:47 UTC
Message-ID: <xmqqwq4xdpgg.fsf@gitster.dls.corp.google.com>
In-Reply-To: <1420739863-11602-1-git-send-email-kuleshovmail@gmail.com>

Alexander Kuleshov <kuleshovmail@gmail.com> writes:

> git remote show doesn't use -v/--verbose option
>
> Signed-off-by: Alexander Kuleshov <kuleshovmail@gmail.com>

Thanks.

I think these two patches should be squashed into one (which I can do locally without asking you to resend) but they are good changes. The subcommand does not just "not use", but it does not even support (i.e. it throws an error when the option is given).

Show 17 quoted lines

> ---
>  builtin/remote.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/builtin/remote.c b/builtin/remote.c
> index 46ecfd9..978c645 100644
> --- a/builtin/remote.c
> +++ b/builtin/remote.c
> @@ -14,7 +14,7 @@ static const char * const builtin_remote_usage[] = {
>  	N_("git remote rename <old> <new>"),
>  	N_("git remote remove <name>"),
>  	N_("git remote set-head <name> (-a | --auto | -d | --delete |<branch>)"),
> -	N_("git remote [-v | --verbose] show [-n] <name>"),
> +	N_("git remote show [-n] <name>"),
>  	N_("git remote prune [-n | --dry-run] <name>"),
>  	N_("git remote [-v | --verbose] update [-p | --prune] [(<group> | <remote>)...]"),
>  	N_("git remote set-branches [--add] <name> <branch>..."),