Re: [PATCH] ls-remote.txt: delete unsupported option

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Nguyễn Thái Ngọc Duy  <pclouds@xxxxxxxxx> writes:

> -u <exec> has never been supported, but it was mentioned since
> 0a2bb55 (git ls-remote: make usage string match manpage -
> 2008-11-11). Nobody has complained about it for seven years, it's
> probably safe to say nobody cares. So let's remove "-u" in documents
> instead of adding code to support it.

This sounds OK.

A potential argument for consistency with fetch does not even apply,
as over there --upload-pack and -u are different options.



>
> While at there, fix --upload-pack syntax too.
>
> Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx>
> ---
>  Documentation/git-ls-remote.txt | 3 +--
>  builtin/ls-remote.c             | 2 +-
>  2 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/git-ls-remote.txt b/Documentation/git-ls-remote.txt
> index 2e22915..d510c05 100644
> --- a/Documentation/git-ls-remote.txt
> +++ b/Documentation/git-ls-remote.txt
> @@ -9,7 +9,7 @@ git-ls-remote - List references in a remote repository
>  SYNOPSIS
>  --------
>  [verse]
> -'git ls-remote' [--heads] [--tags]  [-u <exec> | --upload-pack <exec>]
> +'git ls-remote' [--heads] [--tags]  [--upload-pack=<exec>]
>  	      [--exit-code] <repository> [<refs>...]
>  
>  DESCRIPTION
> @@ -29,7 +29,6 @@ OPTIONS
>  	both, references stored in refs/heads and refs/tags are
>  	displayed.
>  
> --u <exec>::
>  --upload-pack=<exec>::
>  	Specify the full path of 'git-upload-pack' on the remote
>  	host. This allows listing references from repositories accessed via
> diff --git a/builtin/ls-remote.c b/builtin/ls-remote.c
> index 4554dbc..5e9d545 100644
> --- a/builtin/ls-remote.c
> +++ b/builtin/ls-remote.c
> @@ -4,7 +4,7 @@
>  #include "remote.h"
>  
>  static const char ls_remote_usage[] =
> -"git ls-remote [--heads] [--tags]  [-u <exec> | --upload-pack <exec>]\n"
> +"git ls-remote [--heads] [--tags]  [--upload-pack=<exec>]\n"
>  "                     [-q | --quiet] [--exit-code] [--get-url] [<repository> [<refs>...]]";
>  
>  /*
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]