On Sat, 8 Mar 2008, Linus Torvalds wrote:
> On Tue, 4 Mar 2008, Jeff King wrote:
> >
> > I am tracking down a bug in unpack_trees, but I can't seem to find the
> > exact problem; I'm hoping to get help from people who have touched this
> > code a bit more than I have.
>
> Ok, so I decided that I should now finally go back and look at the
> original bug-report that triggered my unpack-trees rewrite, now that it's
> in a form where I feel like I can actually look at the code and fix the
> problem..
>
> I'd love to say that I know what the original bug was, but since I
> couldn't fix it in the first place because I couldn't read the original
> code, I can't really say what fixed it.
The original bug was that the position in the index being modified in
place got messed up by core code that discarded unnecessary REMOVE entries
for files in a d/f conflicting directory without reporting how many were
removed so that the iteration could compensate. Cleaning up the code may
or may not have fixed it, but using separate indices would make it really
hard to retain the bug.
> Jeff's test-script appended just for people who can't find the original
> message that started this all.
Here it is as an actual test case:
----------
commit f9eef3140fedaa10842d433e6fbf67f6b914712c
Author: Daniel Barkalow <barkalow@xxxxxxxxxxxx>
Date: Wed Mar 5 15:50:36 2008 -0500
Add a test for read-tree -u --reset working despite df conflicts
From an email by Jeff King <peff@xxxxxxxx>
Signed-off-by: Daniel Barkalow <barkalow@xxxxxxxxxxxx>
diff --git a/t/t1005-read-tree-reset.sh b/t/t1005-read-tree-reset.sh
new file mode 100755
index 0000000..f1b1216
--- /dev/null
+++ b/t/t1005-read-tree-reset.sh
@@ -0,0 +1,30 @@
+#!/bin/sh
+
+test_description='read-tree -u --reset'
+
+. ./test-lib.sh
+
+# two-tree test
+
+test_expect_success 'setup' '
+ git init &&
+ mkdir df &&
+ echo content >df/file &&
+ git add df/file &&
+ git commit -m one &&
+ git ls-files >expect &&
+ rm -rf df &&
+ echo content >df &&
+ git add df &&
+ echo content >new &&
+ git add new &&
+ git commit -m two
+'
+
+test_expect_failure 'reset should work' '
+ git read-tree -u --reset HEAD^ &&
+ git ls-files >actual &&
+ diff -u expect actual
+'
+
+test_done
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
