darcs

Patch 935 Resolve issue 2226: make sure to do patch index creation last

Title Resolve issue 2226: make sure to do patch index creation last
Superseder Nosy List bsrkaditya
Related Issues make sure to do patch index creation last
View: 2226
Status accepted Assigned To
Milestone

Created on 2012-09-09.14:58:24 by bsrkaditya, last changed 2012-11-12.01:31:05 by owst.

Files
File name Status Uploaded Type Edit Remove
move-patch-index-creation-on-existing-repositories-to-the-end-of-command-execution.dpatch bsrkaditya, 2012-09-09.14:58:24 application/octet-stream
move-patch-index-creation-on-existing-repositories-to-the-end-of-command-execution.dpatch ganesh, 2012-09-09.15:32:59 application/x-darcs-patch
patch-preview.txt ganesh, 2012-09-09.15:32:59 text/x-darcs-patch
unnamed ganesh, 2012-09-09.15:32:59
See mailing list archives for discussion on individual patches.
Messages
msg16066 (view) Author: bsrkaditya Date: 2012-09-09.14:58:24
Patch index is created at the end of command execution.(Unless the 
command is change or annotate)
Attachments
msg16068 (view) Author: ganesh Date: 2012-09-09.15:32:59
just adding a conflict resolution

2 patches for repository darcs-unstable@darcs.net:screened:

Sun Sep  9 15:53:16 BST 2012  bsrkaditya@gmail.com
  * Move patch index creation on existing repositories to the end of command execution
      The exception to this is when the command run is
      changes or annotate, where patch index is created before
      command execution

Sun Sep  9 16:30:05 BST 2012  Ganesh Sittampalam <ganesh@earth.li>
  * resolve conflicts between patch index creation and Darcs.Path renaming
Attachments
msg16070 (view) Author: owst Date: 2012-09-09.15:41:29
This change makes darcs only update an existing PI when finalizing repo
changes (which is quick, relative to a full PI creation, which would
happen if the PI was missing). A missing PI will now only be created
right at the end of the command run.

Looks good, thanks!
msg16071 (view) Author: darcswatch Date: 2012-09-09.15:46:09
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-bbd0fd9eb7ac8fbe69a9d4464f6295cb8654e87a
msg16087 (view) Author: darcswatch Date: 2012-09-12.21:25:31
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-bbd0fd9eb7ac8fbe69a9d4464f6295cb8654e87a
History
Date User Action Args
2012-09-09 14:58:24bsrkadityacreate
2012-09-09 15:32:59ganeshsetfiles: + patch-preview.txt, move-patch-index-creation-on-existing-repositories-to-the-end-of-command-execution.dpatch, unnamed
messages: + msg16068
2012-09-09 15:41:29owstsetstatus: needs-screening -> accepted
messages: + msg16070
2012-09-09 15:44:40owstsetstatus: accepted -> accepted-pending-tests
2012-09-09 15:46:09darcswatchsetmessages: + msg16071
2012-09-12 21:25:31darcswatchsetstatus: accepted-pending-tests -> accepted
messages: + msg16087
2012-11-11 20:44:38ganeshsetstatus: accepted -> accepted-pending-tests
2012-11-12 01:31:05owstsetstatus: accepted-pending-tests -> accepted