diff options
author | Junio C Hamano <gitster@pobox.com> | 2015-06-23 10:27:47 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2015-06-23 10:37:21 -0700 |
commit | 78cc1a540ba127b13f2f3fd531777b57f3a9cd46 (patch) | |
tree | 9f4d294accbb456727dad9dc231afc862d535433 /builtin | |
parent | d95d728aba06a34394d15466045cbdabdada58a2 (diff) | |
download | git-78cc1a540ba127b13f2f3fd531777b57f3a9cd46.tar.gz git-78cc1a540ba127b13f2f3fd531777b57f3a9cd46.tar.xz |
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 <gitster@pobox.com>
Diffstat (limited to 'builtin')
-rw-r--r-- | builtin/add.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/builtin/add.c b/builtin/add.c index ee370b059..3390933d6 100644 --- a/builtin/add.c +++ b/builtin/add.c @@ -63,7 +63,6 @@ static void update_callback(struct diff_queue_struct *q, switch (fix_unmerged_status(p, data)) { default: die(_("unexpected diff status %c"), p->status); - case DIFF_STATUS_ADDED: case DIFF_STATUS_MODIFIED: case DIFF_STATUS_TYPE_CHANGED: if (add_file_to_index(&the_index, path, data->flags)) { |