diff options
author | Thomas Gummerer <t.gummerer@gmail.com> | 2019-09-11 19:20:27 +0100 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2019-09-20 09:58:22 -0700 |
commit | 34933d0eff5d4c91fae6ad6f71a6e6a69a496ced (patch) | |
tree | 4e101712a6452418d6878b11189a488a0c56afe6 /t/t7418-submodule-sparse-gitmodules.sh | |
parent | merge: use refresh_and_write_cache (diff) | |
download | tgif-34933d0eff5d4c91fae6ad6f71a6e6a69a496ced.tar.xz |
stash: make sure to write refreshed cache
When converting stash into C, calls to 'git update-index --refresh'
were replaced with the 'refresh_cache()' function. That is fine as
long as the index is only needed in-core, and not re-read from disk.
However in many cases we do actually need the refreshed index to be
written to disk, for example 'merge_recursive_generic()' discards the
in-core index before re-reading it from disk, and in the case of 'apply
--quiet', the 'refresh_cache()' we currently have is pointless without
writing the index to disk.
Always write the index after refreshing it to ensure there are no
regressions in this compared to the scripted stash. In the future we
can consider avoiding the write where possible after making sure none
of the subsequent calls actually need the refreshed cache, and it is
not expected to be refreshed after stash exits or it is written
somewhere else already.
Reported-by: Jeff King <peff@peff.net>
Signed-off-by: Thomas Gummerer <t.gummerer@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t7418-submodule-sparse-gitmodules.sh')
0 files changed, 0 insertions, 0 deletions