From b741291c708e5c2088680a0d9d3c75efd98db421 Mon Sep 17 00:00:00 2001 From: Karen Arutyunov Date: Tue, 1 Nov 2016 23:31:27 +0300 Subject: Make rmdir_r() to call entry_exists(path) rather than exists(dir_path) --- build2/test/script/runner.cxx | 11 ----------- 1 file changed, 11 deletions(-) (limited to 'build2/test/script/runner.cxx') diff --git a/build2/test/script/runner.cxx b/build2/test/script/runner.cxx index 2af5865..79201b2 100644 --- a/build2/test/script/runner.cxx +++ b/build2/test/script/runner.cxx @@ -254,17 +254,6 @@ namespace build2 // Recursive removal of non-existing directory is not an error for // 'maybe' cleanup type. // - // Note that if some file system entry of non-directory type exists - // with such a name it is not removed but the operation still - // succeeds for 'maybe' cleanup type. The removal of this entry can - // be handled at the time of the containing directory cleanup. - // - // @@ The behavior in the situation described differes for &?a/*** - // and &?a/ due to build2::rmdir_r() implementation details which - // checks for directory existence before trying to remove it. - // Shouldn't rmdir_r() behave the same way as rmdir() in regards - // to non-directory removal? - // if (p.leaf ().string () == "***") { verify (p.directory (), p, "wildcard"); -- cgit v1.1