darcs

Issue 1393 Swap "darcs mv" and "darcs move".

Title Swap "darcs mv" and "darcs move".
Priority wishlist Status resolved
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, kowey, thorkilnaur, twb
Assigned To
Topics

Created on 2009-03-15.06:40:00 by twb, last changed 2009-08-27.14:16:09 by admin.

Files
File name Uploaded Type Edit Remove
leverage-existing-command_alias-macro_.dpatch twb, 2009-03-15.08:11:22 text/x-darcs-patch
unnamed twb, 2009-03-15.08:11:22 text/plain
Messages
msg7458 (view) Author: twb Date: 2009-03-15.06:39:58
Currently, "darcs move" is an alias for the command "darcs mv".  As I
think "darcs move" is clearer, it should be the command and "darcs mv"
should be the alias.  AFAICT that change will not break anything.
msg7459 (view) Author: twb Date: 2009-03-15.08:11:22
Sun Mar 15 17:53:33 EST 2009  Trent W. Buck <trentbuck@gmail.com>
  * Leverage existing command_alias macro.

Sun Mar 15 19:05:02 EST 2009  Trent W. Buck <trentbuck@gmail.com>
  * Resolve issue1393: swap "darcs mv" and "darcs move".
Attachments
msg7460 (view) Author: kowey Date: 2009-03-15.09:08:03
So I don't have any strong feelings on the matter, but I think the argument was
that darcs mv actually effectuates a move in the working directory (hence a name
that resembles the Unix command), whereas darcs remove does not (hence a
different name).

Do we care about that?
msg7462 (view) Author: twb Date: 2009-03-15.09:34:19
Ultimately, what this change will do is mean that in "darcs help" and
the manpage/manual, "move" will appear instead of "mv".

On Sun, Mar 15, 2009 at 09:08:07AM -0000, Eric Kow wrote:
> So I don't have any strong feelings on the matter, but I think the
> argument was that darcs mv actually effectuates a move in the
> working directory (hence a name that resembles the Unix command),
> whereas darcs remove does not (hence a different name).

Correct, and the patch supplied removes a comment to that effect.

> Do we care about that?

Though I'm a Unix user, I don't consider that an adequate reason for
"mv" to be the default.  A non-negligible portion of our userbase are
on Windows and (presumably) do not think in terms of "mv".  When a
user is looking through "darcs help", I think even Unix users will
generally look for "move" before they look for "mv".  And if a Unix
user intuitively types "darcs mv", it will still work (because mv and
move stay synonymous).
msg7463 (view) Author: kowey Date: 2009-03-15.12:30:29
Ok, so I will tell darcs-users that I intend to apply this, and that they have
24 hours to complain if they disagree.  (Trying to balance between being
sufficiently consultative and not inviting too many bikeshed painting exercises).
msg7476 (view) Author: twb Date: 2009-03-17.09:32:51
The following patch updated the status of issue1393 to be resolved:

* Resolve issue1393: swap "darcs mv" and "darcs move". 
Ignore-this: 773dc2c5a53c67839b5b19ceedbc7236
History
Date User Action Args
2009-03-15 06:40:00twbcreate
2009-03-15 08:11:24twbsetfiles: + leverage-existing-command_alias-macro_.dpatch, unnamed
nosy: kowey, simon, twb, thorkilnaur, dmitry.kurochkin
status: unread -> unknown
messages: + msg7459
2009-03-15 09:08:06koweysetpriority: wishlist
nosy: kowey, simon, twb, thorkilnaur, dmitry.kurochkin
messages: + msg7460
2009-03-15 09:34:22twbsetnosy: kowey, simon, twb, thorkilnaur, dmitry.kurochkin
messages: + msg7462
2009-03-15 12:30:31koweysetnosy: kowey, simon, twb, thorkilnaur, dmitry.kurochkin
messages: + msg7463
2009-03-17 09:32:54twbsetstatus: unknown -> resolved
nosy: kowey, simon, twb, thorkilnaur, dmitry.kurochkin
messages: + msg7476
2009-08-25 17:42:08adminsetnosy: + darcs-devel, - simon
2009-08-27 14:16:09adminsetnosy: kowey, darcs-devel, twb, thorkilnaur, dmitry.kurochkin