path: root/t/
diff options
authorJunio C Hamano <>2015-06-23 17:27:47 (GMT)
committerJunio C Hamano <>2015-06-23 17:37:21 (GMT)
commit78cc1a540ba127b13f2f3fd531777b57f3a9cd46 (patch)
tree9f4d294accbb456727dad9dc231afc862d535433 /t/
parentd95d728aba06a34394d15466045cbdabdada58a2 (diff)
Revert "diff-lib.c: adjust position of i-t-a entries in diff"
This reverts commit d95d728aba06a34394d15466045cbdabdada58a2. It turns out that many other commands that need to interact with the result of running diff-files and diff-index, e.g. "git apply", "git rm", etc., need to be adjusted to the new world order it brings in. For example, it would break this sequence to correct a whitespace breakage in the parts you changed: git add -N file git diff --cached file | git apply --cached --whitespace=fix git checkout file In the old world order, "diff" showed a patch to modify an existing empty file by adding its full contents, and "apply" updated the index by modifying the existing empty blob (which is what an Intent-to-Add entry records in the index) with that patch. In the new world order, "diff" shows a patch to create a new file with its full contents, but because "apply" thinks that the i-t-a entry already exists in the index, it refused to accept a creation. Adjusting "apply" to this new world order is easy, but we need to assess the extent of the damage to the rest of the system the new world order brought in before going forward and adjust them all, after which we can resurrect the commit being reverted here. Signed-off-by: Junio C Hamano <>
Diffstat (limited to 't/')
1 files changed, 4 insertions, 19 deletions
diff --git a/t/ b/t/
index 7c641bf..2a4a749 100755
--- a/t/
+++ b/t/
@@ -5,24 +5,10 @@ test_description='Intent to add'
. ./
test_expect_success 'intent to add' '
- test_commit 1 &&
- git rm 1.t &&
- echo hello >1.t &&
echo hello >file &&
echo hello >elif &&
git add -N file &&
- git add elif &&
- git add -N 1.t
-test_expect_success 'git status' '
- git status --porcelain | grep -v actual >actual &&
- cat >expect <<-\EOF &&
- DA 1.t
- A elif
- A file
- test_cmp expect actual
+ git add elif
test_expect_success 'check result of "add -N"' '
@@ -57,8 +43,7 @@ test_expect_success 'i-t-a entry is simply ignored' '
git add -N nitfol &&
git commit -m second &&
test $(git ls-tree HEAD -- nitfol | wc -l) = 0 &&
- test $(git diff --name-only HEAD -- nitfol | wc -l) = 0 &&
- test $(git diff --name-only -- nitfol | wc -l) = 1
+ test $(git diff --name-only HEAD -- nitfol | wc -l) = 1
test_expect_success 'can commit with an unrelated i-t-a entry in index' '
@@ -87,13 +72,13 @@ test_expect_success 'cache-tree invalidates i-t-a paths' '
: >dir/bar &&
git add -N dir/bar &&
git diff --cached --name-only >actual &&
- >expect &&
+ echo dir/bar >expect &&
test_cmp expect actual &&
git write-tree >/dev/null &&
git diff --cached --name-only >actual &&
- >expect &&
+ echo dir/bar >expect &&
test_cmp expect actual