darcs

Issue 2531 help for non-existing subcommand should indicate failure

Title help for non-existing subcommand should indicate failure
Priority Status resolved
Milestone Resolved in
Superseder Nosy List bfrk, jaredj
Assigned To
Topics ProbablyEasy

Created on 2017-03-26.15:44:53 by bfrk, last changed 2023-07-16.20:17:33 by bfrk.

Files
File name Uploaded Type Edit Remove
resolve-issue2531_-help-for-non_existing-subcommand-should-fail.dpatch bfrk, 2017-08-12.15:12:18 application/x-darcs-patch
Messages
msg19403 (view) Author: bfrk Date: 2017-03-26.15:44:52
The help text displayed is the same as for the super command (show).
msg19404 (view) Author: gh Date: 2017-03-26.16:47:45
Looks like it's actually another bug.

"darcs help show patch-index" is the command you want to use

"darcs help show patchindex" or "darcs help show blabla" should not
work. But they currently show the same output as "darcs help show".
They should probably fail instead.

2017-03-26 12:44 GMT-03:00 Ben Franksen <bugs@darcs.net>:
>
> New submission from Ben Franksen <ben.franksen@online.de>:
>
> The help text displayed is the same as for the super command (show).
>
> ----------
> messages: 19403
> nosy: bf
> status: unknown
> title: no specific help for show patchIndex
>
> __________________________________
> Darcs bug tracker <bugs@darcs.net>
> <http://bugs.darcs.net/issue2531>
> __________________________________
> _______________________________________________
> darcs-devel mailing list
> darcs-devel@darcs.net
> http://lists.osuosl.org/mailman/listinfo/darcs-devel
msg19405 (view) Author: bfrk Date: 2017-03-26.19:00:31
> Guillaume Hoffmann <guillaumh@gmail.com> added the comment:
> 
> Looks like it's actually another bug.
> 
> "darcs help show patch-index" is the command you want to use
> 
> "darcs help show patchindex" or "darcs help show blabla" should not
> work. But they currently show the same output as "darcs help show".
> They should probably fail instead.

No idea what made me think it would be 'show patchIndex'. Perhaps it was
the name of the option in the code.
msg23599 (view) Author: bfrk Date: 2023-07-16.20:17:32
Resolved, at least in the head:

ben@home[4]:~>darcs help show bla | head -5
Invalid subcommand!

Usage: darcs show SUBCOMMAND [OPTION]...
Show information about the given repository.
History
Date User Action Args
2017-03-26 15:44:53bfrkcreate
2017-03-26 16:47:47ghsetmessages: + msg19404
2017-03-26 19:00:32bfrksetmessages: + msg19405
2017-03-27 00:25:03bfrksettitle: no specific help for show patchIndex -> help for non-existing subcommand should indicate failure
2017-07-31 02:25:11ghsettopic: + ProbablyEasy
nosy: + jaredj
2017-08-12 15:12:19bfrksetfiles: + resolve-issue2531_-help-for-non_existing-subcommand-should-fail.dpatch
2023-07-16 20:17:33bfrksetstatus: unknown -> resolved
messages: + msg23599