Created on 2009-03-15.06:40:00 by twb, last changed 2009-08-27.14:16:09 by admin.
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
|
|
Date |
User |
Action |
Args |
2009-03-15 06:40:00 | twb | create | |
2009-03-15 08:11:24 | twb | set | files:
+ leverage-existing-command_alias-macro_.dpatch, unnamed nosy:
kowey, simon, twb, thorkilnaur, dmitry.kurochkin status: unread -> unknown messages:
+ msg7459 |
2009-03-15 09:08:06 | kowey | set | priority: wishlist nosy:
kowey, simon, twb, thorkilnaur, dmitry.kurochkin messages:
+ msg7460 |
2009-03-15 09:34:22 | twb | set | nosy:
kowey, simon, twb, thorkilnaur, dmitry.kurochkin messages:
+ msg7462 |
2009-03-15 12:30:31 | kowey | set | nosy:
kowey, simon, twb, thorkilnaur, dmitry.kurochkin messages:
+ msg7463 |
2009-03-17 09:32:54 | twb | set | status: unknown -> resolved nosy:
kowey, simon, twb, thorkilnaur, dmitry.kurochkin messages:
+ msg7476 |
2009-08-25 17:42:08 | admin | set | nosy:
+ darcs-devel, - simon |
2009-08-27 14:16:09 | admin | set | nosy:
kowey, darcs-devel, twb, thorkilnaur, dmitry.kurochkin |
|