Jeff King <peff@xxxxxxxx> writes: > On Wed, Jul 02, 2014 at 12:01:59PM -0700, Jonathan Nieder wrote: > >> Jeff King wrote: >> >> > Signed-off-by: Jeff King <peff@xxxxxxxx> >> > --- >> > t/t7300-clean.sh | 4 ++-- >> > 1 file changed, 2 insertions(+), 2 deletions(-) >> >> Does the later "git clean -d with an unreadable empty directory" test >> need the same treatment? > > I don't think so, because it is also failing during the experiments I'm > doing and it is not causing the same problem. It tests that "clean -d" is happy if a blind rmdir(2) removes the directory. If it fails for whatever reason (e.g. add "exit(0)" at the beginning of cmd_clean(), for example) to remove the directory, we do leave an empty unreadable directory behind. But as long as that directory is empty, that will not cause us to remove the trash directory at the end, so we should be OK. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html