[darcs-devel] darcs patch: add test for replace that messes with un... (and 3 more)

[David Roundy]

Hi all,

Here's a sort of random-looking set of patches.  Well, just one's
random-looking.  Mostly, this is a fix to the bug that Tommy revealed in
his patch to replace.sh.  But that was hard to track down, and involved
complicated issues with regard to --ignore-times, which inspired a rather
extensive (but safe) cleanup, in which I added a [DarcsFlag] field to
Repository.  The result is that now a Repository "knows" if it should be
ignoring the time stamps or not.  This refactoring could lead to others,
since there are certainly functions that should no longer need to have the
[DarcsFlag] passed.  It also may sink the idea of putting [DarcsFlag] in a
global variable (which was always just a bit ugly).

David

Thu Jan 25 07:38:03 PST 2007  Tommy Pettersson <ptp at lysator.liu.se>
  * add test for replace that messes with unrecorded hunks

Sat Jan 27 16:07:28 PST 2007  David Roundy <droundy at darcs.net>
  * refactor: add opts into Repository.

Sat Jan 27 16:18:26 PST 2007  David Roundy <droundy at darcs.net>
  * fix bugs in replace.sh script--running wrong darcs.

Sat Jan 27 16:22:06 PST 2007  David Roundy <droundy at darcs.net>
  * fix bug triggered in replace.sh
  This bug was an annoying one that seemed to involve trouble caused by
  unsafeInterleaveIO and the order of evaluation, since we change the working
  directory.  I've simplified the code significantly.  Complicating the debug
  process was a race condition caused by the lack of --ignore-times in
  replace.sh, which was because darcs replace didn't accept that option.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: text/x-darcs-patch
Size: 83606 bytes
Desc: A darcs patch for your repository!
Url : http://lists.osuosl.org/pipermail/darcs-devel/attachments/20070127/6ae407b9/attachment-0001.bin