Created on 2006-04-10.19:42:23 by markstos, last changed 2009-08-27.14:04:17 by admin.
msg615 (view) |
Author: markstos |
Date: 2006-04-10.19:42:20 |
|
( One more 'rollback' related bug import for today. This one is from RT#459).
####
If I try to rollback a depended-upon patch, darcs just complains:
--- clip ---
$ darcs changes
Wed Jul 6 14:12:49 EEST 2005 atehwa@sange.fi
* changed a line in foo
Wed Jul 6 14:12:32 EEST 2005 atehwa@sange.fi
* added a line into foo
Wed Jul 6 14:12:04 EEST 2005 atehwa@sange.fi
* adding foo
$ darcs rollback -p 'added a line into foo'
Skipping depended-upon patch:
Wed Jul 6 14:12:32 EEST 2005 atehwa@sange.fi
* added a line into foo
Cancelling rollback since no patch was selected.
--- clip ---
I think darcs should, instead, try to raise the patch as near to the
current moment as possible without conflicts, and then reverse-apply it.
# Thu Oct 06 17:02:00 2005 jch - Correspondence added
> If I try to rollback a depended-upon patch, darcs just complains:
> I think darcs should, instead, try to raise the patch as near to the
> current moment as possible without conflicts, and then reverse-apply it.
I agree. More precisely, the behaviour of rolling back A in (A,B) should
be the same as merging (A, B) with (A, A-1) (i.e. causing conflicts to
happen).
|
msg2558 (view) |
Author: markstos |
Date: 2008-01-18.03:13:43 |
|
Even with the re-written rollback, it isn't possible to rollback a depended-upon
patch, unless the right sequence of patches in the dependency chain is also
selected to rollback.
I can't decide whether it even makes sense to support trying to a rollback a
patch with the dependency chain for it, but for people who really want to, it
seems the UI could be improved to either list what the dependency chain is, or
work sort of "apt-get":
"Selecting this patch for rollback implies that the following patches which also
depend on it must also be rolled-back. Would you like to proceed?"
Anyway, this issue remains at "wishlist" status, so I would say other
improvements to darcs are higher priority, unless this is easy to address further.
|
msg2755 (view) |
Author: droundy |
Date: 2008-01-25.20:22:45 |
|
On Fri, Jan 18, 2008 at 03:13:45AM -0000, Mark Stosberg wrote:
> Even with the re-written rollback, it isn't possible to rollback a depended-upon
> patch, unless the right sequence of patches in the dependency chain is also
> selected to rollback.
I'll just point out that since we can rollback only a subset of the changes
in the selected patches, it's now possible (but awkward) to get the effect
of rolling back only a selected patch and the primitive patches that depend
on it.
And selecting the dependencies is actually quite easy simply by hitting 'w'
(or 'j') until you reach the patch you want to roll back.
David
|
msg3308 (view) |
Author: markstos |
Date: 2008-02-11.01:34:13 |
|
Based on David's last comment it sounds like this "wish" is addressed somewhat
in the unstable branch, so I'm marking this as "resolved-in-unstable".
Please re-test with a Darcs 2 build before considering re-opening the ticket.
|
|
Date |
User |
Action |
Args |
2006-04-10 19:42:23 | markstos | create | |
2006-05-04 17:30:59 | jch | set | nosy:
droundy, tommy, markstos |
2006-05-04 17:33:48 | jch | set | priority: feature -> wishlist nosy:
droundy, tommy, markstos |
2008-01-18 03:13:45 | markstos | set | status: unread -> unknown nosy:
+ beschmi, kowey messages:
+ msg2558 |
2008-01-25 20:22:46 | droundy | set | nosy:
droundy, tommy, beschmi, kowey, markstos messages:
+ msg2755 |
2008-02-11 01:34:14 | markstos | set | status: unknown -> resolved-in-unstable nosy:
droundy, tommy, beschmi, kowey, markstos messages:
+ msg3308 |
2008-09-04 21:28:17 | admin | set | status: resolved-in-unstable -> resolved nosy:
+ dagit |
2009-08-06 17:41:11 | admin | set | nosy:
+ jast, Serware, dmitry.kurochkin, darcs-devel, zooko, mornfall, simon, thorkilnaur, - droundy |
2009-08-06 20:49:32 | admin | set | nosy:
- beschmi |
2009-08-10 21:44:54 | admin | set | nosy:
- darcs-devel, zooko, jast, Serware, mornfall |
2009-08-10 23:53:51 | admin | set | nosy:
- dagit |
2009-08-25 17:54:45 | admin | set | nosy:
+ darcs-devel, - simon |
2009-08-27 14:04:17 | admin | set | nosy:
tommy, kowey, markstos, darcs-devel, thorkilnaur, dmitry.kurochkin |
|