diff options
| author | Alex Riesen <raa.lkml@gmail.com> | 2011-04-01 08:29:16 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2011-04-01 18:16:21 (GMT) |
| commit | 0235017eafa8dfc3a7353b7091ad5109d0beb7a5 (patch) | |
| tree | f62dcba6b55c4043600bbd0620b91e54c885b129 | |
| parent | 2352570bf43368f3d3931f7b24bb7fd44e68eef3 (diff) | |
| download | git-0235017eafa8dfc3a7353b7091ad5109d0beb7a5.zip git-0235017eafa8dfc3a7353b7091ad5109d0beb7a5.tar.gz git-0235017eafa8dfc3a7353b7091ad5109d0beb7a5.tar.bz2 | |
clean: unreadable directory may still be rmdir-able if it is empty
As a last ditch effort, try rmdir(2) when we cannot read the directory
to be removed. It may be an empty directory that we can remove without
any permission, as long as we can modify its parent directory.
Noticed by Linus.
Signed-off-by: Alex Riesen <raa.lkml@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | dir.c | 2 | ||||
| -rwxr-xr-x | t/t7300-clean.sh | 7 |
2 files changed, 8 insertions, 1 deletions
@@ -1088,7 +1088,7 @@ int remove_dir_recursively(struct strbuf *path, int flag) dir = opendir(path->buf); if (!dir) - return -1; + return rmdir(path->buf); if (path->buf[original_len - 1] != '/') strbuf_addch(path, '/'); diff --git a/t/t7300-clean.sh b/t/t7300-clean.sh index 02f67b7..9b12681 100755 --- a/t/t7300-clean.sh +++ b/t/t7300-clean.sh @@ -453,4 +453,11 @@ test_expect_success 'git clean -e' ' ) ' +test_expect_success SANITY 'git clean -d with an unreadable empty directory' ' + mkdir foo && + chmod a= foo && + git clean -dfx foo && + ! test -d foo +' + test_done |