darcs

Issue 2531 help for non-existing subcommand should indicate failure

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

Created on 2017-03-26.15:44:53 by bf, last changed 2017-08-12.15:12:19 by bf.

Files
File name Uploaded Type Edit Remove
resolve-issue2531_-help-for-non_existing-subcommand-should-fail.dpatch bf, 2017-08-12.15:12:18 application/x-darcs-patch
Messages
msg19403 (view) Author: bf 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: bf 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.
History
Date User Action Args
2017-03-26 15:44:53bfcreate
2017-03-26 16:47:47ghsetmessages: + msg19404
2017-03-26 19:00:32bfsetmessages: + msg19405
2017-03-27 00:25:03bfsettitle: 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:19bfsetfiles: + resolve-issue2531_-help-for-non_existing-subcommand-should-fail.dpatch