1. Summarise the issue (what were doing, what went wrong?)

darcs amend-un<TAB> doesn't complete

2. What behaviour were you expecting instead?

That my zsh (or bash) completions know about the amend-unrecord alias.

3. What darcs version are you using? (Try: darcs --exact-version)

HEAD of screened repo

4. What operating system are you running?

linux

A bit more rationale.

I find amend-unrecord very useful and use it regularly. It is less
verbose that amend-record --unrecord and the name feels quite intuitive
to me. Listing it with --commands would give it more visibility which
would be a good thing IMO.

For what it's worth, it was a deliberate decision to hide it in order to 
preserve the default "amend" shorthand for "amend-record" - otherwise 
that would become ambiguous.

I'm happy to revisit that decision, just want to make the reason clear. 
We could also make the defaulting logic customisable so that amend 
remained as that abbreviation.

Indeed there is a visibility problem with the --unrecord flag, but I'm
not fond of the solution proposed by Ben. Instead of adding an extra
top-level commands (which I believe we would collectively resist), I'd
rather enhance how things are shown.

I'm attaching an alternative proposal that:

* lists --unrecord first in the help of amend
* trims obvious/unnecessary text from the help of amend and mention
--unrecord at the second paragraph
* renames amend-record to amend (leaves amend-record as alias)
* removes the amend-unrecord alias

The last two items seemed natural to me in that context. "amend" (or
"amend --add") is the default mode and "amend --unrecord" is the
substractive mode.

Try "darcs help amend" with that patch.

Attachments

• rename-amend_record-to-amend_-make-__unrecord-more-visible-in-help.dpatch

I understand (and appreciate) that you guys want to keep the user
interface clean. GH's proposal is very nice and better than mine. 

Except for the naming of the default switch: --add should rather be
--record. Then we'd have

  darcs amend-record == darcs amend --record

while --add would be confusing since 'darcs add' does something
completely unrelated.

Here is a patch which changes --add to --record and slightly improves on
the wording of the option help text.

Attachments

• rename-amend_record_plus1.dpatch

I like it :) let's see if there are more reactions, and if there are
not (and you agree) I'll submit both patches in a couple of days.

2014-04-06 21:42 GMT-03:00 Ben Franksen <bugs@darcs.net>:
>
> Ben Franksen <benjamin.franksen@helmholtz-berlin.de> added the comment:
>
> Here is a patch which changes --add to --record and slightly improves on
> the wording of the option help text.
>
> __________________________________
> Darcs bug tracker <bugs@darcs.net>
> <http://bugs.darcs.net/issue2367>
> __________________________________
>
> _______________________________________________
> darcs-devel mailing list
> darcs-devel@darcs.net
> http://lists.osuosl.org/mailman/listinfo/darcs-devel
>

I'm updating the title of the issue to reflect the current proposal.

The following patch sent by Guillaume Hoffmann <guillaumh@gmail.com> updated issue issue2367 with
status=resolved;resolvedin=2.10.0 HEAD

* resolve issue2367: rename amend-record to amend, make --unrecord more visible 
Ignore-this: 6da507ce5415b45b56dd0c81f988bdb3
also rename amend flag --add to --record