darcs

Patch 456 patch442

Title patch442
Superseder Nosy List gh
Related Issues
Status rejected Assigned To
Milestone

Created on 2010-11-09.16:44:46 by gh, last changed 2010-11-15.14:40:10 by kowey. Tracked on DarcsWatch.

Files
File name Status Uploaded Type Edit Remove
remove-all-tests-related-to-checkpoints-and-__partial.dpatch gh, 2010-11-09.16:44:46 text/x-darcs-patch
unnamed gh, 2010-11-09.16:44:46
See mailing list archives for discussion on individual patches.
Messages
msg12964 (view) Author: gh Date: 2010-11-09.16:44:46
Reinier, you are right. I'm new to dividing patches to enhance commutation,
so I only separated the warning-removing patch (2) from the feature-removing
one (1) because (2) modifies lines that may easily be touched by other
patches (the import/export lines at beginning of files). In case of heavy
modification of import lines by other patches, (1) may still be applied
without conflicts. But yes it is better to split the (1) patch, there
was indeed one conflict because of that!


The attached new bundle applies cleanly to the current screened branch.


4 patches for repository http://darcs.net/screened:

Mon Nov  1 11:03:24 CET 2010  Guillaume Hoffmann <guillaumh@gmail.com>
  * remove all tests related to checkpoints and --partial

Mon Nov  1 11:16:02 CET 2010  Guillaume Hoffmann <guillaumh@gmail.com>
  * remove get --ephemeral support

Mon Nov  8 20:11:37 CET 2010  Guillaume Hoffmann <guillaumh@gmail.com>
  * remove support for get --partial and check --partial

Mon Nov  8 20:13:10 CET 2010  Guillaume Hoffmann <guillaumh@gmail.com>
  * remove Darcs.Repository.Checkpoint
Attachments
msg13062 (view) Author: kowey Date: 2010-11-15.14:40:10
This was sent in error, according to Guillaume, so I'm rejecting. Thanks 
:-)
History
Date User Action Args
2010-11-09 16:44:46ghcreate
2010-11-09 16:47:29darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-93a3a52bbc361e8ce680afc0cf038c37f017dc8d
2010-11-15 14:40:10koweysetstatus: needs-screening -> rejected
messages: + msg13062