darcs

Issue 279 bug in get_extra commuting patch ** minimal test case **

Title bug in get_extra commuting patch ** minimal test case **
Priority bug Status resolved
Milestone Resolved in
Superseder put with --darcs-2 => "Malformed patch bundle"
View: 661
Nosy List andrew1, darcs-devel, dmitry.kurochkin, kowey, markstos, thorkilnaur, tommy
Assigned To
Topics Conflicts

Created on 2006-09-29.23:14:31 by andrew1, last changed 2009-08-27.13:58:04 by admin.

Messages
msg1025 (view) Author: andrew1 Date: 2006-09-29.23:14:22
Below is the transcript of a bug that comes from pulling the resolution
of a three-way conflict onto a four-way conflict (the three-way conflict
plus another).  Specifically, there are four conflicting patches a, b,
c, and d.  The first three are combined in repo c, and then resolved by
patch rc.  All four are combined in repo d, then we try to pull rc from
repo c.

Andrew

~% mkdir 0
~% cd 0
~/0% darcs init
~/0% echo 0 > f           
~/0% darcs add f
~/0% darcs record -a -m 0 
Finished recording patch '0'
~/0% cd ..
~% for r in a b c d; do darcs get 0 $r; (cd $r; echo $r > f; darcs record --ignore-times -a -m $r); done
Copying patch 1 of 1... done!
Finished getting.
Finished recording patch 'a'
Copying patch 1 of 1... done!
Finished getting.
Finished recording patch 'b'
Copying patch 1 of 1... done!
Finished getting.
Finished recording patch 'c'
Copying patch 1 of 1... done!
Finished getting.
Finished recording patch 'd'
~% cd d
~/d% darcs pull -a ../a
We have conflicts in the following files:
./f
Finished pulling and applying.
~/d% darcs pull -a ../b
We have conflicts in the following files:
./f
Finished pulling and applying.
~/d% darcs pull -a ../c
We have conflicts in the following files:
./f
Finished pulling and applying.
~/d% cd ../c
~/c% darcs pull -a ../a
We have conflicts in the following files:
./f
Finished pulling and applying.
~/c% darcs pull -a ../b
We have conflicts in the following files:
./f
Finished pulling and applying.
~/c% echo rc > f
~/c% darcs record -a -m rc
Finished recording patch 'rc'
~/c% cd ../d
~/d% darcs pull -a ../c
Pulling from "../c"...
darcs: bug in get_extra commuting patch:
Fri Sep 29 15:37:15 PDT 2006  andrew.pimlott.ctr@metnet.navy.mil
  * d
zsh: exit 1     /home/andrew/tmp/darcs-unstable/darcs pull -a ../c
msg1035 (view) Author: kowey Date: 2006-10-01.20:47:50
Hi Andrew!

On Fri, Sep 29, 2006 at 23:14:31 +0000, Andrew Pimlott wrote:
> Below is the transcript of a bug that comes from pulling the resolution
> of a three-way conflict onto a four-way conflict (the three-way conflict
> plus another).

Thanks for the bug report and especially the transcript.

I'm going to merge and defer this one until we solve issue175, which has
the same error message.  Will definitely note the transcript.
msg2036 (view) Author: kowey Date: 2007-08-10.09:17:38
Note Tommy's message to darcs-devel on 2007-08-09

  http://article.gmane.org/gmane.comp.version-control.darcs.devel/6300

It has sort of a workaround (while waiting for the new patch theory stuff to
come out)
msg3296 (view) Author: markstos Date: 2008-02-10.02:23:57
I just created an automated test for this and submitted it to the bugs/ directory. 

With the old fashioned repo format, it still fails as described above. I suspect
it would pass with the --darcs-2 format, but instead fails with a "Malformed
patch bundle" message, which I believe is separate bug that we already have a
test for. I'm marking the "Malformed patch" bug as a superseder for now.
msg3299 (view) Author: markstos Date: 2008-02-10.02:32:02
On second thought, I'm going to go ahead and consider this
"resolved-in-unstable", since it appears to be a dupe of other bugs that are. 

If bugs/issue279.pl continues to fail once the Malformed Patch bug is fixed and
the --darcs-2 format is used, then we'll know we need to re-open this.
msg4429 (view) Author: kowey Date: 2008-05-01.16:03:22
Marking as resolved. issue279.pl has been moved to the passing tests/ directory,
and darcs2 has long been released.
History
Date User Action Args
2006-09-29 23:14:31andrew1create
2006-10-01 20:47:55koweysetstatus: unread -> deferred
nosy: droundy, tommy, kowey, andrew1
superseder: + obliterate --tags => bug in get_extra commuting patch (1.07)
messages: + msg1035
2007-03-08 11:26:24koweysetnosy: + beschmi
2007-07-23 20:30:16koweysettopic: + Conflicts
nosy: droundy, tommy, beschmi, kowey, andrew1
title: bug in get_extra commuting patch -> bug in get_extra commuting patch ** minimal test case **
2007-08-03 17:55:50koweylinkissue175 superseder
2007-08-03 17:56:02koweysetstatus: deferred -> unknown
superseder: - obliterate --tags => bug in get_extra commuting patch (1.07)
2007-08-08 08:09:46koweylinkissue428 superseder
2007-08-10 09:17:39koweysetmessages: + msg2036
2008-02-10 02:23:58markstossetnosy: + markstos
superseder: + put with --darcs-2 => "Malformed patch bundle"
messages: + msg3296
2008-02-10 02:32:03markstossetstatus: unknown -> resolved-in-unstable
nosy: droundy, tommy, beschmi, kowey, markstos, andrew1
messages: + msg3299
2008-05-01 16:03:25koweysetstatus: resolved-in-unstable -> resolved
nosy: + dagit
messages: + msg4429
2009-08-06 17:47:41adminsetnosy: + jast, Serware, dmitry.kurochkin, darcs-devel, zooko, mornfall, simon, thorkilnaur, - droundy, andrew1
2009-08-06 20:43:28adminsetnosy: - beschmi
2009-08-10 21:56:07adminsetnosy: + andrew1, - darcs-devel, zooko, jast, Serware, mornfall
2009-08-10 23:56:26adminsetnosy: - dagit
2009-08-25 17:59:26adminsetnosy: + darcs-devel, - simon
2009-08-27 13:58:04adminsetnosy: tommy, kowey, markstos, darcs-devel, andrew1, thorkilnaur, dmitry.kurochkin