On Tue, Jan 02, 2018 at 03:04:43PM -0500, Jeff King wrote: > So I don't think there's an urgent data-loss bug here; we will only ever > destroy an empty directory. However, the original intent was to leave > the filesystem as we found it on a failed or aborted clone, and we > obviously don't do that in this case. So it might be nice if we could > restore it to an empty directory. Here's a patch series to do that. The first three are just preparatory cleanups; the last one is the interesting bit. [1/4]: t5600: fix outdated comment about unborn HEAD [2/4]: t5600: modernize style [3/4]: clone: factor out dir_exists() helper [4/4]: clone: do not clean up directories we didn't create builtin/clone.c | 31 ++++++++++--- t/t5600-clone-fail-cleanup.sh | 100 +++++++++++++++++++++++++++++++----------- 2 files changed, 98 insertions(+), 33 deletions(-) -Peff
