[PATCH] request-pull: drop old USAGE stuff

[PATCH] request-pull: drop old USAGE stuff

[Wolfram Sang]

request-pull uses OPTIONS_SPEC, so no need for (meanwhile incomplete)
USAGE and LONG_USAGE anymore.

Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
---

 git-request-pull.sh |    3 ---
 1 file changed, 3 deletions(-)

Index: git-2.11.0/git-request-pull.sh
===================================================================
--- git-2.11.0.orig/git-request-pull.sh
+++ git-2.11.0/git-request-pull.sh
@@ -4,9 +4,6 @@
 # This file is licensed under the GPL v2, or a later version
 # at the discretion of Linus Torvalds.
 
-USAGE='<start> <url> [<end>]'
-LONG_USAGE='Summarizes the changes between two commits to the standard output,
-and includes the given URL in the generated summary.'
 SUBDIRECTORY_OK='Yes'
 OPTIONS_KEEPDASHDASH=
 OPTIONS_STUCKLONG=

Re: [PATCH] request-pull: drop old USAGE stuff

[Junio C Hamano]

Wolfram Sang <wsa@the-dreams.de> writes:

> request-pull uses OPTIONS_SPEC, so no need for (meanwhile incomplete)
> USAGE and LONG_USAGE anymore.
>
> Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
> ---

Makes sense.  These are not used anywhere after we switched to use
parse-options.

Thanks.

>
>  git-request-pull.sh |    3 ---
>  1 file changed, 3 deletions(-)
>
> Index: git-2.11.0/git-request-pull.sh
> ===================================================================
> --- git-2.11.0.orig/git-request-pull.sh
> +++ git-2.11.0/git-request-pull.sh
> @@ -4,9 +4,6 @@
>  # This file is licensed under the GPL v2, or a later version
>  # at the discretion of Linus Torvalds.
>  
> -USAGE='<start> <url> [<end>]'
> -LONG_USAGE='Summarizes the changes between two commits to the standard output,
> -and includes the given URL in the generated summary.'
>  SUBDIRECTORY_OK='Yes'
>  OPTIONS_KEEPDASHDASH=
>  OPTIONS_STUCKLONG=

Re: [PATCH] request-pull: drop old USAGE stuff

[Jeff King]

On Sun, Jan 15, 2017 at 04:23:02PM -0800, Junio C Hamano wrote:

> Wolfram Sang <wsa@the-dreams.de> writes:
> 
> > request-pull uses OPTIONS_SPEC, so no need for (meanwhile incomplete)
> > USAGE and LONG_USAGE anymore.
> >
> > Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
> > ---
> 
> Makes sense.  These are not used anywhere after we switched to use
> parse-options.

It does seem a shame that parse-options does not show this explanatory
text. But I guess nobody really cares that much, and you can always use
"--help" to get even more details.

-Peff