Patch 45: Make --reverse flag available for all patch selection commands - Darcs bug tracker

Title	Make --reverse flag available for all patch selection commands
Superseder		Nosy List	dagit, darcs-users, galbolle, ganesh
Related Issues
Status	rejected	Assigned To
Milestone

Created on 2009-11-13.15:02:56 by galbolle, last changed 2009-12-05.17:45:21 by ganesh.

Files
File name	Status	Uploaded	Type	Edit	Remove
add-__dont_ignore_times-counterpart-to-__ignore_times.dpatch		galbolle, 2009-11-13.15:02:51	text/x-darcs-patch
unnamed		galbolle, 2009-11-13.15:02:51	text/plain
unnamed		dagit, 2009-11-20.22:38:40	text/html

See mailing list archives for discussion on individual patches.

Messages
msg9242 (view)	Author: galbolle	Date: 2009-11-13.15:02:52
As an example of usage, this bundle was created by calling darcs send --reverse and answering y then d. This is much more darcsish than using 'w' until the wanted patch appears, or using --match. Wed Nov 11 17:32:50 CET 2009 Florent Becker <florent.becker@ens-lyon.org> * Add --dont-ignore-times counterpart to --ignore-times Wed Nov 11 18:27:39 CET 2009 Florent Becker <florent.becker@ens-lyon.org> * add --dont-delete-log-file option Wed Nov 11 19:23:01 CET 2009 Florent Becker <florent.becker@ens-lyon.org> * add a --traditional counterpart to --unified Wed Nov 11 20:28:04 CET 2009 Florent Becker <florent.becker@ens-lyon.org> * add --apply-on-disk as counterpart to --store-in-memory Wed Nov 11 20:54:43 CET 2009 Florent Becker <florent.becker@ens-lyon.org> * add --no-happy-forwarding Mon Nov 2 17:55:51 CET 2009 Florent Becker <florent.becker@ens-lyon.org> * Refactor Darcs.Commands.Pull Fri Nov 13 15:33:46 CET 2009 Florent Becker <florent.becker@ens-lyon.org> * add --reverse to all patch-selecting commands For instance, darcs send --reverse allows a sometimes more natural goal-driven choice of patches: "goal" patches are offered first, then their dependencies are added automatically. Attachments add-__dont_ignore_times-counterpart-to-__ignore_times.dpatch unnamed
msg9394 (view)	Author: galbolle	Date: 2009-11-16.15:25:15
Some seemingly unrelated patches appear because of dependency on part of patch43 .
msg9435 (view)	Author: galbolle	Date: 2009-11-20.22:13:32
This patch is probably wrong, even if I have not yet been able to find a counter-example. The preselection of patches (patches_to_consider) in the Last and FirstReversed cases seems fishy (it was not used before). Tips by anyone who understands SelectChanges welcome.
msg9437 (view)	Author: dagit	Date: 2009-11-20.22:38:40
On Fri, Nov 20, 2009 at 2:13 PM, Florent Becker <bugs@darcs.net> wrote: > > Florent Becker <florent.becker@ens-lyon.org> added the comment: > > This patch is probably wrong, even if I have not yet been able to find a > counter-example. The preselection of patches (patches_to_consider) in the > Last > and FirstReversed cases seems fishy (it was not used before). Tips by > anyone who > understands SelectChanges welcome. > I have created Issue1698 for the task of improving the SelectChanges code. Attachments unnamed
msg9534 (view)	Author: ganesh	Date: 2009-12-05.17:45:21
Marking as rejected at Florent's suggestion; he's going to resubmit after cleaning up SelectChanges.

History
Date	User	Action	Args
2009-11-13 15:02:56	galbolle	create
2009-11-16 15:25:15	galbolle	set	messages: + msg9394 title: Add --dont-ignore-times counterpart to -... (and 6 more) -> Make --reverse flag available for all patch selection commands
2009-11-17 10:26:11	kowey	set	nosy: + darcs-users
2009-11-20 22:13:33	galbolle	set	messages: + msg9435
2009-11-20 22:38:41	dagit	set	files: + unnamed nosy: + dagit messages: + msg9437
2009-12-05 17:45:21	ganesh	set	status: needs-review -> rejected nosy: + ganesh messages: + msg9534