The following script results in the error message "darcs: bug in get_extra
commuting patches:".  This is the bug I mentioned in
http://lists.osuosl.org/pipermail/darcs-users/2009-March/018548.html

The setup is very similar to the one in Issue1327, but the error message is
different.


mkdir repo1
cd repo1
darcs init
echo fileA version 1 > fileA
echo fileB version 1 > fileB
darcs add fileA fileB
darcs record --author foo@bar --ignore-times --all -m "Add fileA and fileB"
echo fileA version 2 > fileA
darcs record --author foo@bar --ignore-times --all -m "Modify fileA"
cd ..
darcs get repo1 repo2
cd repo2
darcs obliterate -p "Modify fileA" --all
darcs unrecord -p "Add fileA and fileB" --all
darcs record --author foo@bar --ignore-times --all fileA -m "Add just fileA"
darcs revert --all
darcs pull --all
echo y | darcs obliterate --dont-prompt-for-dependencies -p "Add fileA and fileB"
darcs pull

I've added this to bugs/ in the Darcs repo.

twb: Have you?  When I pull from darcs.net I don't see it.

On Sun, Mar 29, 2009 at 09:42:54PM -0000, Aaron Kaplan wrote:
> twb: Have you?  When I pull from darcs.net I don't see it.

Sorry, it has been delayed by our review process.  I need to amend the
patch and resubmit it.  I expect to have it done sometime this week.

Just an update: this is now in the test suite.

I suspect that this is a duplicate of issue1014.  I say for the superficial
reason of having seen duplicate patches in the example (adding f in one repo,
then adding f and g in another repo).

I would need to somebody to scrutinise this and maybe boil the two tests down
even more.  Ian: is there any chance you could have a quick look at the
issue1401 bug and confirm if this is just yet another variant or something else?

One "empirical" approach might be to check against David's Darcs too since it
fixes 1014

Looks like Ian is busy.  

In issue1014, you create duplicate patches A and B, then patch C which
depends on A (or B).  If you create a repo with just BC and try to pull
A from ABC, darcs blows up.

This looks a bit different because now you're unpulling things, but
because there are Darcs 2 duplicate patch shenanigans going on, I'm
going to work on the assumption that this is the same bug.  If we fix
isuse1014, we should revisit this.

(Note that David Roundy's fork of Darcs has an issue1014 fix; it'd be a
good idea to see if it fixes this as well).

This looks like the same bug to me, yes.


Thanks
Ian