I would like to hear opinions about this: is this a bug or a feature? do we
want the unrevert bundle contain conflicts?

1 patch for repository http://darcs.net/screened:

patch fe565b18beaf9169a28b8c49464904a0f27e0f5a
Author: Ben Franksen <ben.franksen@online.de>
Date:   Thu Nov 15 15:18:07 CET 2018
  * accept issue2609: conflicts in unrevert bundle

Attachments

• accept-issue2609_-conflicts-in-unrevert-bundle.dpatch
• patch-preview.txt
• unnamed

Feels like a feature to me, but I'm not certain.

I find it interesting that this test fails with darcs-3 patch format, 
too, even though the underlying scenario (issue2605) no longer does.

I should read my own comments. Nothing to do with issue2605, see 
msg20465. Also conflict is not in the unrevert bundle but appears when 
we apply it, see msg20839. So I guess this is a false alarm and I 
should rollback.

My view is that unrevert conflicting with recorded is not
very surprising, and that unrevert containing conflicts may or may
not be valid.

Whenever either scenario can arise, presumably the alternative is 
just to throw it away. So I'd be inclined to let it happen.

On the other hand if the only way you can cause unrevert to contain
conflicts is with the duplicate patch hack like in msg20460, then
I almost don't care what happens because I think that hack is 
already taking us into very messy territory and this is the least
of the problems.

There is no connection with duplicate patches, i just stumbled over this
behavior when experimenting with them. The conflict is not /in/ the unrevert
bundle, it happens due to unrevert conflicting with recorded. This is
normal, expected behavior. Instead of rolling back, I thought it better to
turn it into a succeeding test, documenting the expected behavior.

1 patch for repository http://darcs.net/screened:

patch 604ff48230dd0c6ef74216f3d9c8ea856a4436c4
Author: Ben Franksen <ben.franksen@online.de>
Date:   Sun Jun 16 12:16:32 CEST 2019
  * issue2609 is not a bug but expected behavior

Attachments

• issue2609-is-not-a-bug-but-expected-behavior.dpatch
• patch-preview.txt
• unnamed