darcs

Patch 164 Resolve issue182: hidden darcs status alias for darcs ...

Title Resolve issue182: hidden darcs status alias for darcs ...
Superseder Nosy List galbolle, ganesh, kowey
Related Issues
Status accepted Assigned To ganesh
Milestone

Created on 2010-02-19.15:51:05 by kowey, last changed 2011-05-10.21:06:06 by darcswatch. Tracked on DarcsWatch.

Files
File name Status Uploaded Type Edit Remove
resolve-issue182_-hidden-darcs-status-alias-for-darcs-whatsnew-_ls.dpatch kowey, 2010-02-19.15:51:02 text/x-darcs-patch
resolve-issue182_-hidden-darcs-status-alias-for-darcs-whatsnew-_ls.dpatch kowey, 2010-09-15.21:04:17 text/x-darcs-patch
unnamed kowey, 2010-02-19.15:51:02 text/plain
unnamed kowey, 2010-09-15.21:04:17
See mailing list archives for discussion on individual patches.
Messages
msg10035 (view) Author: kowey Date: 2010-02-19.15:51:02
1 patch for repository http://darcs.net

This is motivated by:
 - http://community.livejournal.com/evan_tech/270665.html
 - http://wiki.darcs.net/RosettaStone
 
Note that this would really only be resolved if http://bugs.darcs.net/issue1725
were fixed.  Stopping recursion into untracked directories would make the
output a good deal more readable.  The end result may still not exactly match
other VCS, but I don't think that's important.  The key is that darcs status
should do something useful and somewhat recognisable.

Whoever reviews this, I would welcome a grumpy response along the lines of "we
should avoid command/alias proliferation" even for hidden commands; or that
such a patch should not be accepted without discussion.

I wouldn't be sure how to respond to that, however, as I'm operating
exclusively on the short-term level of giving folks what they think they want
(which isn't always what they really want), and not on the long-term wiser
level of thinking about what the consequences of this will be.

Fri Feb 19 15:32:38 GMT 2010  Eric Kow <kowey@darcs.net>
  * Resolve issue182: hidden darcs status alias for darcs whatsnew -ls
Attachments
msg12542 (view) Author: kowey Date: 2010-09-14.20:05:41
Mind reviewing this, Florent?  I got stuck trying to implement a little
(darcs whatsnew -s) notification, which just keeps getting chewed up by
round tuit shortage... also it's a suggestion that should really be made
separately with its own thinking.  I think we should just get this
"darcs status" thing unstuck
msg12566 (view) Author: ganesh Date: 2010-09-15.20:22:32
This now conflicts with HEAD.
msg12570 (view) Author: kowey Date: 2010-09-15.21:04:17
1 patch for repository http://darcs.net/screened:

This is a very old patch in before the screening days, so I took the liberty of
resending it as an amended patch, rather than patch plus conflict resolution.

Maybe old habits just die hard.  I consider this sane enough to be pre-screened.

Wed Sep 15 22:05:19 BST 2010  Eric Kow <kowey@darcs.net>
  * Resolve issue182: hidden darcs status alias for darcs whatsnew -ls
Attachments
msg12572 (view) Author: simon Date: 2010-09-15.21:08:14
On 9/14/10 1:05 PM, Eric Kow wrote:
> Mind reviewing this, Florent?  I got stuck trying to implement a little
> (darcs whatsnew -s) notification, which just keeps getting chewed up by
> round tuit shortage... also it's a suggestion that should really be made
> separately with its own thinking.  I think we should just get this
> "darcs status" thing unstuck

I'm one of those "no gratuitous aliases!" types, or at least "let's have a policy, and document them".

Pro: "status" looks like it would smooth the way for a lot of new/infrequent darcs users. And it's consistent with the 
current de facto policy since we already have these hidden aliases: unadd, rm, mv, commit, resolve, log, list, query, 
transferMode, unpull, clone, gzcrcs.

Con: you don't want to have to maintain docs/tests/examples covering both commands. There'd be a chance that "whatsnew" 
gets diluted or deprecated at some point down the road.

The code changes look simple and good.

So I guess I'm a +0.9 for this one.
msg12674 (view) Author: ganesh Date: 2010-10-09.16:08:43
Grabbing as it's been on the review queue for a while. I think it looks 
reasonable. Anyone have any objections before I apply it?

Of course, a test would be nice :-)
msg12823 (view) Author: darcswatch Date: 2010-10-24.20:28:06
This patch bundle (with 1 patches) was just applied to the repository http://darcs.net/.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-b3497273f04934a04ac38936130ac963c0271983
msg14028 (view) Author: darcswatch Date: 2011-05-10.17:15:52
This patch bundle (with 1 patches) was just applied to the repository http://darcs.net/reviewed.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_reviewed.html#bundle-b3497273f04934a04ac38936130ac963c0271983
History
Date User Action Args
2010-02-19 15:51:05koweycreate
2010-02-19 15:54:12darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-0cb177477e4ec7e0ee5f9133b93eb6eac722540b
2010-03-15 11:39:23koweysetstatus: needs-review -> followup-requested
assignedto: kowey
2010-06-24 12:50:19koweysetstatus: followup-requested -> followup-in-progress
2010-09-14 20:05:41koweysetstatus: followup-in-progress -> needs-review
nosy: + galbolle, - darcs-users
messages: + msg12542
assignedto: kowey -> galbolle
2010-09-15 10:42:12koweysetstatus: needs-review -> needs-screening
2010-09-15 20:22:32ganeshsetstatus: needs-screening -> followup-requested
messages: + msg12566
2010-09-15 21:04:17koweysetfiles: + resolve-issue182_-hidden-darcs-status-alias-for-darcs-whatsnew-_ls.dpatch, unnamed
messages: + msg12570
2010-09-15 21:04:46darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-0cb177477e4ec7e0ee5f9133b93eb6eac722540b -> http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-b3497273f04934a04ac38936130ac963c0271983
2010-09-15 21:05:41koweysetstatus: followup-requested -> needs-review
2010-09-15 21:08:15simonsetmessages: + msg12572
2010-10-09 16:08:43ganeshsetstatus: needs-review -> review-in-progress
nosy: + ganesh
messages: + msg12674
assignedto: galbolle -> ganesh
2010-10-24 20:28:07darcswatchsetstatus: review-in-progress -> accepted
messages: + msg12823
2011-05-10 17:15:52darcswatchsetmessages: + msg14028
2011-05-10 21:06:06darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-b3497273f04934a04ac38936130ac963c0271983 -> http://darcswatch.nomeata.de/repo_http:__darcs.net_reviewed.html#bundle-0cb177477e4ec7e0ee5f9133b93eb6eac722540b