BTW, my opinion on this is that being able to replace an implicit dependency
is a very nice and useful feature. Fixing this bug should be easy.

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

patch e58e96882a706d6a854029b393f1a58c47aa2bbf
Author: Ben Franksen <ben.franksen@online.de>
Date:   Sun Oct 28 14:47:18 CET 2018
  * accept issue2605: darcs crashes with duplicate patches

Attachments

• accept-issue2605_-darcs-crashes-with-duplicate-patches.dpatch
• patch-preview.txt
• unnamed

This test isn't marked as failing which means screened is broken, is
that intentional?

No that wasn't intentional, sorry. I can send a rename patch and 
push it and then rebase the bundle that fixes the issue. Or I can 
push the fix, which I haven't done yet. I would prefer the latter.

The reason I didn't push it yet is to give you a chance to comment 
on the general direction this goes. The fix changes the semantics of 
merging from by-name to by-value/content. I thought this may provoke 
dissent even though the underlying commute machinery has always been 
by-value and I am pretty sure it is impossible to fix this bug in 
any other way except by adding prim patch identifiers (as proposed 
long ago by Ian Lynagh).

Thanks, I've got a rename locally, I'll push that for now.

I'll comment on the fix once I've had time to collect my thoughts
properly (hopefully within the next couple of days).

1 patch for repository darcs-unstable@darcs.net:screened:

patch ecc7509b36b8ee98a535226c9e2f880cc549dc91
Author: Ganesh Sittampalam <ganesh@earth.li>
Date:   Sun Nov 18 11:10:30 GMT 2018
  * issue2605 test is currently failing

Attachments

• issue2605-test-is-currently-failing.dpatch
• patch-preview.txt
• unnamed