[darcs-users] reviewing and testing pulls

David Roundy droundy at abridgegame.org
Tue Apr 27 12:53:51 UTC 2004


(I think your other questions are answered elsewhere in this thread, if
not, just reask...)

On Mon, Apr 26, 2004 at 05:38:30PM -0400, Andrew Pimlott wrote:
> 1b. Why isn't there a nice human-readable form of a patch-set?
> 
> Has it just not been done yet?  I realize that a darcs patch can't be
> represented as a simple diff, but is it too much to hope for something
> diff-like?  It doesn't have to be usable for applying the patches, just
> for reviewing them.  "darcs send" could optionally include both forms in
> an email.

Would darcs whatsnew -u be similar to what you'd want? It's not perfect
when there are hunks close to one another (the context gets weird in that
case), but isn't bad either.  It's a bit more of a pain to output patches
with context, but is definitely doable, and patches with context are still
valid darcs patches (darcs currently just ignores the context, although it
could be used for redundant checks).  Adding -u support to send is
something I've wanted to do, but never got around to.

> 2.  How do I revert a whole pull?
> 
> One alternative to the above is to pull, then use "darcs diff".  (An
> annoyance here is that you have to pass to -m the patch _before_ the
> first pulled; it would be nice to have an option that includes the patch
> specified.)  However, there is no single command to undo the whole pull.
> 
> Although I could use a separate tree, and throw it away to undo the
> pull, it would be nice for darcs to remember which patches were added in
> the same batch, and let me unpull the whole batch.

To me, this doesn't seem like it would be a very useful feature.  I'd say
that if you are likely to want to unpull a bunch of patches that you pulled
all at once, you should pull into a temporary repo.  Of course, you may not
know beforehand that you won't want the changes, but it doesn't seem *that*
hard to just run unpull a bunch of times.

Basically, since there are already arguably too many ways to undo patches,
I'd hesitate before adding any more ways to undo changes.
-- 
David Roundy
http://www.abridgegame.org




More information about the darcs-users mailing list