diff options
author | David Aguilar <davvid@gmail.com> | 2016-12-07 02:16:08 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2016-12-08 10:28:00 -0800 |
commit | 853e10c19755669bd4695b1d9fd81f3278466727 (patch) | |
tree | a3c1ee83cc6444cea3de3fc272b03165359f65d3 /string-list.c | |
parent | Prepare for 2.9.4 (diff) | |
download | tgif-853e10c19755669bd4695b1d9fd81f3278466727.tar.xz |
difftool: fix dir-diff index creation when in a subdirectory
9ec26e7977 (difftool: fix argument handling in subdirs, 2016-07-18)
corrected how path arguments are handled in a subdirectory, but
it introduced a regression in how entries outside of the
subdirectory are handled by dir-diff.
When preparing the right-side of the diff we only include the
changed paths in the temporary area.
The left side of the diff is constructed from a temporary
index that is built from the same set of changed files, but it
was being constructed from within the subdirectory. This is a
problem because the indexed paths are toplevel-relative, and
thus they were not getting added to the index.
Teach difftool to chdir to the toplevel of the repository before
preparing its temporary indexes. This ensures that all of the
toplevel-relative paths are valid.
Add test cases to more thoroughly exercise this scenario.
Reported-by: Frank Becker <fb@mooflu.com>
Signed-off-by: David Aguilar <davvid@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'string-list.c')
0 files changed, 0 insertions, 0 deletions