Patch 431: Support hunk editing in rollback - Darcs bug tracker

Title	Support hunk editing in rollback
Superseder		Nosy List	galbolle, kowey
Related Issues
Status	accepted	Assigned To
Milestone

Created on 2010-10-20.10:19:40 by galbolle, last changed 2011-05-10.19:06:22 by darcswatch. Tracked on DarcsWatch.

Files
File name	Status	Uploaded	Type	Edit	Remove
support-hunk-editing-in-rollback.dpatch		galbolle, 2010-10-20.10:19:40	text/x-darcs-patch
unnamed		galbolle, 2010-10-20.10:19:40

See mailing list archives for discussion on individual patches.

Messages
msg12784 (view)	Author: galbolle	Date: 2010-10-20.10:19:40
This adds 'e' in rollback, which I found useful at least once. It's awkward to use right now, since given: ==== Before (in the patch) line i want to rollback to be kept ==== After (with the rollback) to be kept ==== the first question darcs asks is: hunk ./a 1 +to be kept Shall I rollback this change? (2/3) [ynW...], or ? for more options: to which the answer is 'n', whereas in record, the first question has to be answered with 'y'. Editing the kept line in addition leads to puzzlement. Interface suggestions welcome 1 patch for repository http://darcs.net/screened: Tue Oct 19 19:07:06 CEST 2010 Florent Becker <florent.becker@ens-lyon.org> * Support hunk editing in rollback Attachments support-hunk-editing-in-rollback.dpatch unnamed
msg12889 (view)	Author: darcswatch	Date: 2010-11-02.23:30:13
This patch bundle (with 1 patches) was just applied to the repository http://darcs.net/screened. This message was brought to you by DarcsWatch http://darcswatch.nomeata.de/repo_http:__darcs.net_screened.html#bundle-7c579d29622316e6ad6b8999f075a9161dec46fe
msg13575 (view)	Author: gh	Date: 2011-01-24.15:50:04
I have tested it: it does not crash. I can not find a use for it but of Florent needed it, then it's useful. The good thing is that this patch is really just 3 lines of code, so it's easily undo'able if we discover this is a bad idea. I'm pushing it.
msg13576 (view)	Author: darcswatch	Date: 2011-01-24.15:55:53
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-7c579d29622316e6ad6b8999f075a9161dec46fe
msg14153 (view)	Author: darcswatch	Date: 2011-05-10.19:06:22
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-7c579d29622316e6ad6b8999f075a9161dec46fe

History
Date	User	Action	Args
2010-10-20 10:19:40	galbolle	create
2010-10-20 10:20:35	darcswatch	set	darcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-7c579d29622316e6ad6b8999f075a9161dec46fe
2010-11-02 23:30:13	darcswatch	set	status: needs-screening -> needs-review messages: + msg12889
2010-12-15 17:42:49	kowey	set	status: needs-review -> followup-requested
2010-12-15 17:43:12	kowey	set	status: followup-requested -> (no value)
2010-12-15 17:43:19	kowey	set	status: needs-review
2010-12-15 17:54:38	kowey	set	status: needs-review -> followup-requested assignedto: galbolle
2010-12-15 17:54:51	kowey	set	status: followup-requested -> needs-review assignedto: galbolle -> (no value)
2010-12-15 17:55:38	kowey	set	assignedto: kowey nosy: + kowey
2010-12-15 17:55:47	kowey	set	status: needs-review -> followup-requested assignedto: kowey -> galbolle
2010-12-15 17:56:01	kowey	set	status: followup-requested -> needs-review
2010-12-15 17:56:10	kowey	set	status: needs-review -> followup-requested assignedto: galbolle -> kowey
2010-12-15 17:56:18	kowey	set	status: followup-requested -> needs-review assignedto: kowey ->
2011-01-24 15:50:04	gh	set	messages: + msg13575
2011-01-24 15:55:53	darcswatch	set	status: needs-review -> accepted messages: + msg13576
2011-05-10 19:06:22	darcswatch	set	messages: + msg14153