Created on 2006-07-03.20:25:38 by adamfine, last changed 2009-08-27.13:52:20 by admin.
msg736 (view) |
Author: adamfine |
Date: 2006-07-03.20:25:35 |
|
Hi
I noticed "darcs rollback" lacks a --last=NUMBER optional argument,
the sort that unrecord et. al. have.
Could have used it today, and I think it's generally a nice argument
and useful for rollback in particular.
Regrads,
Adam
|
msg2302 (view) |
Author: markstos |
Date: 2008-01-01.22:17:47 |
|
darcs rollback is designed to select a single patch, which is why "--last" is
not an option.
I'm fine with leaving it that way and resolving this as "wont-fix".
David, Eric: You have both touched this ticket. What do you think about rollback
selecting more than one patch?
|
msg2303 (view) |
Author: kowey |
Date: 2008-01-01.22:56:29 |
|
> David, Eric: You have both touched this ticket. What do you think about rollback
> selecting more than one patch?
Well, I don't have a very clear picture in my head of how --last is
really supposed to behave, but I figured that since rollback accepts
--match and --patch, --last would be a pretty reasonable addition as
well.
The only thing that is affected is which patches darcs proposes to
you. It is still only one patch that gets rolled back, but having
these matchers might help to narrow down the set of patches to choose
from.
darcs rollback --last=1, for instance, might be a good use for it.
P.S. welcome back! and good luck/happy hacking with the Haskell
|
msg2317 (view) |
Author: droundy |
Date: 2008-01-05.14:40:25 |
|
On Tue, Jan 01, 2008 at 10:56:30PM -0000, Eric Kow wrote:
> > David, Eric: You have both touched this ticket. What do you think about
> > rollback selecting more than one patch?
>
> Well, I don't have a very clear picture in my head of how --last is
> really supposed to behave, but I figured that since rollback accepts
> --match and --patch, --last would be a pretty reasonable addition as
> well.
>
> The only thing that is affected is which patches darcs proposes to
> you. It is still only one patch that gets rolled back, but having
> these matchers might help to narrow down the set of patches to choose
> from.
>
> darcs rollback --last=1, for instance, might be a good use for it.
Yeah, that makes sense. Internally, it'd be a bit of a pain, and I think
if we added --last, it'd only be consistent to add --matches and --patches
as well. And I suspect that those might be confusing.
Incidentally, I'm seriously considering removing rollback for darcs 2.
With the way the conflict handling ended up turning out, it's no more
useful than in darcs 1, and it's always been a bad command. :(
--
David Roundy
Department of Physics
Oregon State University
|
msg2330 (view) |
Author: markstos |
Date: 2008-01-05.16:03:32 |
|
David is right that it would be consistent to add --matches and --patches along
with --last, and I don't think appropriate in this case.
I don't think the "--last=1" example is a worthwhile use-case, since the very
last patch is the first one you are offered to rollback in interactive mode.
So, I'm being assertive and marking this as "wont-fix", but anyone who disagrees
is a welcome to re-open the wihlist item.
( Of course, the whole thing will be moot if the rollback is removed. )
|
|
Date |
User |
Action |
Args |
2006-07-03 20:25:38 | adamfine | create | |
2007-08-05 20:21:12 | kowey | set | topic:
+ ProbablyEasy nosy:
+ kowey, beschmi |
2008-01-01 22:17:50 | markstos | set | status: unread -> unknown nosy:
+ markstos messages:
+ msg2302 |
2008-01-01 22:56:31 | kowey | set | messages:
+ msg2303 |
2008-01-05 14:40:28 | droundy | set | messages:
+ msg2317 |
2008-01-05 16:03:33 | markstos | set | status: unknown -> wont-fix nosy:
markstos, adamfine, droundy, tommy, kowey, beschmi messages:
+ msg2330 |
2009-08-06 17:31:57 | admin | set | nosy:
+ jast, Serware, dmitry.kurochkin, darcs-devel, zooko, dagit, mornfall, simon, thorkilnaur, - droundy, adamfine |
2009-08-06 20:28:45 | admin | set | nosy:
- beschmi |
2009-08-10 21:52:44 | admin | set | nosy:
+ adamfine, - darcs-devel, zooko, jast, dagit, Serware, mornfall |
2009-08-25 17:46:43 | admin | set | nosy:
+ darcs-devel, - simon |
2009-08-27 13:52:20 | admin | set | nosy:
tommy, kowey, markstos, darcs-devel, adamfine, thorkilnaur, dmitry.kurochkin |
|