darcs

Issue 1338 darcs unrevert: take files as arguments (like darcs revert)

Title darcs unrevert: take files as arguments (like darcs revert)
Priority wishlist Status needs-implementation
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, gwern, jaredj, kowey, owst, thorkilnaur
Assigned To owst
Topics UI

Created on 2009-01-30.23:29:22 by gwern, last changed 2012-04-12.13:31:32 by owst.

Messages
msg7250 (view) Author: gwern Date: 2009-01-30.23:29:15
So the other day I happened to need to back out some changes in my
xmonad.hs (I've been trying out sjanssen's monoid branch of
xmonad-core), and so I reverted it. Once I was done testing, I
unreverted. Or, at least I tried to.

I had made the mistake of doing 'revert xmonad.hs' and naively tried
to do 'unrevert xmonad.hs'. This didn't work:

 gwern@craft:39361~>revert .xmonad.hs
 Reverting changes in ".xmonad.hs"..

 There are no changes to revert!
 Finished reverting.
 gwern@craft:39360~>unrevert .xmonad.hs

 darcs failed:  Bad argument: `.xmonad.hs'
 Usage: darcs unrevert [OPTION]...
 Undo the last revert (may fail if changes after the revert).

 See darcs help unrevert for details.

(I eventually tried unrevert with no arguments, and had to manually
select the changes to xmonad.hs.) Now, personally, I think unrevert
should take a file argument since it makes just as much sense as for
revert. It is consistent, and desirable from a UI standpoint anyway.
msg15559 (view) Author: owst Date: 2012-04-12.13:31:31
This just hit me too, I'll look into it. The inconsistency between
revert/unrevert is annoying. (Though in my case, I did a "revert -a"
rather than a revert with a path, which is why I needed to selectively
unrevert)
History
Date User Action Args
2009-01-30 23:29:22gwerncreate
2009-01-30 23:30:19twbsettopic: + UI
nosy: kowey, simon, thorkilnaur, gwern, dmitry.kurochkin
2009-01-30 23:30:25twbsetpriority: bug -> wishlist
nosy: kowey, simon, thorkilnaur, gwern, dmitry.kurochkin
2009-03-28 11:26:37twbsetnosy: kowey, simon, thorkilnaur, gwern, dmitry.kurochkin
title: darcs unrevert behavior inconsistent with revert -> darcs unrevert: take files as arguments (like darcs revert)
2009-03-28 11:26:49twbsetnosy: kowey, simon, thorkilnaur, gwern, dmitry.kurochkin
2009-04-09 13:45:51koweysetstatus: unread -> needs-reproduction
nosy: + jaredj
topic: + ProbablyEasy
2009-08-25 17:40:16adminsetnosy: + darcs-devel, - simon
2009-08-27 14:16:37adminsetnosy: kowey, darcs-devel, thorkilnaur, gwern, jaredj, dmitry.kurochkin
2009-09-04 11:26:18koweysetstatus: needs-reproduction -> needs-implementation
nosy: kowey, darcs-devel, thorkilnaur, gwern, jaredj, dmitry.kurochkin
topic: - ProbablyEasy
2012-04-12 13:31:32owstsetassignedto: owst
messages: + msg15559
nosy: + owst