[PATCH 0/1] fix ls-remote help message

STALE3332d

4 messages, 2 authors, 2017-06-28 · open the first message on its own page

[PATCH 0/1] fix ls-remote help message

From: Silvio Fricke <hidden>
Date: 2017-06-27 20:34:32

Hi,

a small patch about a wrong help text. Please include when okay.


Best Regards,
Silvio

Silvio Fricke (1):
  ls-remote: remove "-h" from help text

 builtin/ls-remote.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.13.2

[PATCH 1/1] ls-remote: remove "-h" from help text

From: Silvio Fricke <hidden>
Date: 2017-06-27 20:34:27

This regression was fixed in 91a640ffb6d9 ("ls-remote: a lone "-h" is
asking for help") and the wrong help text was introduced in ba5f28bf79ea
("ls-remote: use parse-options api").
This patch removes the "-h" on the help text.

Signed-off-by: Silvio Fricke <redacted>
---
 builtin/ls-remote.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/builtin/ls-remote.c b/builtin/ls-remote.c
index b2d7d5ce6..cd54f7e98 100644
--- a/builtin/ls-remote.c
+++ b/builtin/ls-remote.c
@@ -56,7 +56,7 @@ int cmd_ls_remote(int argc, const char **argv, const char *prefix)
 			   N_("path of git-upload-pack on the remote host"),
 			   PARSE_OPT_HIDDEN },
 		OPT_BIT('t', "tags", &flags, N_("limit to tags"), REF_TAGS),
-		OPT_BIT('h', "heads", &flags, N_("limit to heads"), REF_HEADS),
+		OPT_BIT(0, "heads", &flags, N_("limit to heads"), REF_HEADS),
 		OPT_BIT(0, "refs", &flags, N_("do not show peeled tags"), REF_NORMAL),
 		OPT_BOOL(0, "get-url", &get_url,
 			 N_("take url.<base>.insteadOf into account")),
-- 
2.13.2

Re: [PATCH 1/1] ls-remote: remove "-h" from help text

From: Jonathan Nieder <hidden>
Date: 2017-06-27 21:09:14

Hi,

Silvio Fricke wrote:
This regression was fixed in 91a640ffb6d9 ("ls-remote: a lone "-h" is
asking for help") and the wrong help text was introduced in ba5f28bf79ea
("ls-remote: use parse-options api").
This patch removes the "-h" on the help text.

Signed-off-by: Silvio Fricke <redacted>
---
 builtin/ls-remote.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Without this patch, I'm able to run

	git ls-remote -h .

This patch removes that support.  Intended?

Thanks and hope that helps,
Jonathan

Re: [PATCH 1/1] ls-remote: remove "-h" from help text

From: S. Fricke <hidden>
Date: 2017-06-28 09:34:43

Hi Jonathan,

quoted
This regression was fixed in 91a640ffb6d9 ("ls-remote: a lone "-h" is
[...]
Without this patch, I'm able to run

	git ls-remote -h .

This patch removes that support.  Intended?
*hihi* okay it was to counter-intuitive for me. "91a640ffb6d" talks about this
issue.

Thanks for enlightenment :-)
Silvio

-- 
-- S. Fricke ---------------------------------------- silvio@port1024.net --
   Diplom-Informatiker (FH)
   Linux-Development                         Matrix: @silvio:port1024.net   
----------------------------------------------------------------------------
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help