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

[David Roundy]

Hi all,

Eric, I think I've addressed at least one of your concerns with my refactor
here.  I haven't addressed those times when identifyRepository is called
with an empty set of options, but those I pretty carefully reviewed when
calling them, so I think it's looking pretty good now.

Incidentally (unrelated to this email), darcs has recently started opening
my emacs without the file open that I want to edit (so I have to find it
with "ls -tr").  Not sure what's triggered this, and I don't have time to
look into it, but hopefully if you all keep an eye out for it, someone will
track it down and fix it.

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: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.

Sun Jan 28 08:58:40 PST 2007  Tommy Pettersson <ptp at lysator.liu.se>
  * use (empty) opts to Repository in list_authors and make_changelog
  They where left out in the opts Repository refactoring.

Sun Jan 28 15:14:05 PST 2007  David Roundy <droundy at darcs.net>
  * refactor get_unrecorded.
  I've removed the [DarcsFlag] argument, and added two new functions
  get_unrecorded_unsorted, and get_unrecorded_no_look_for_adds, which do what
  they say.  I think this simplifies its use, and cleans things up a tad.  It
  doesn't scale to many different ways to get_unrecorded, but I don't think
  we want to go there.

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