darcs

Patch 2258 optimize reorder: create an inventory for each clean tag

Title optimize reorder: create an inventory for each clean tag
Superseder Nosy List bfrk
Related Issues
Status rejected Assigned To
Milestone

Created on 2022-06-13.11:42:46 by bfrk, last changed 2022-06-18.07:52:49 by bfrk.

Files
File name Status Uploaded Type Edit Remove
optimize-reorder_-create-an-inventory-for-each-clean-tag.dpatch bfrk, 2022-06-13.11:42:43 application/x-darcs-patch
patch-preview.txt bfrk, 2022-06-13.11:42:43 text/x-darcs-patch
See mailing list archives for discussion on individual patches.
Messages
msg23025 (view) Author: bfrk Date: 2022-06-13.11:42:43
Ganesh, could you please comment on this i.e. do you think this is a bad
idea? As alternatives, I could create a new command or an option for this.

1 patch for repository http://darcs.net/screened:

patch d2c0e3dccf507ae86a05ca67e8d4bdaeb0b37444
Author: Ben Franksen <ben.franksen@online.de>
Date:   Fri Jun 25 19:36:00 CEST 2021
  * optimize reorder: create an inventory for each clean tag

  This is a change in behavior. Previously, this command only made the latest
  tag clean (and then stored it as an inventory). With this change the
  lazy-optimize-reorder test script no longer makes sense so it has been
  removed.
Attachments
msg23029 (view) Author: ganesh Date: 2022-06-13.20:02:50
> Ganesh, could you please comment on this i.e. do you think this is a
> bad idea? As alternatives, I could create a new command or an option 
> for this.

What's the goal? Feels like the test script is checking for a legitimate 
use case, i.e. reordering "recent" patches without forcing the whole 
repo. So by default I'd be inclined towards an option, but I don't feel
strongly.
msg23031 (view) Author: bfrk Date: 2022-06-18.07:52:49
Ok, will add an option. New patch will also improve 
fullyOptimizePatchset to do some "deep" reorderings.
History
Date User Action Args
2022-06-13 11:42:46bfrkcreate
2022-06-13 20:02:50ganeshsetmessages: + msg23029
2022-06-18 07:52:49bfrksetstatus: needs-screening -> rejected
messages: + msg23031