Merge topic 'revert-install-DIRECTORY-exclude-empty' into release-4.1

21e99db install(DIRECTORY): Revert "Add EXCLUDE_EMPTY_DIRECTORIES option"

Acked-by: Kitware Robot <[email protected]>
Merge-request: !10977

Author: bradking

Help/command/install.rst

@@ -648,7 +648,6 @@ Signatures
             [USE_SOURCE_PERMISSIONS] [OPTIONAL] [MESSAGE_NEVER]
             [CONFIGURATIONS <config>...]
             [COMPONENT <component>] [EXCLUDE_FROM_ALL]
-            [EXCLUDE_EMPTY_DIRECTORIES]
             [FILES_MATCHING]
             [<match-rule> <match-option>...]...
             )
@@ -751,13 +750,6 @@ Signatures
 
     Disable file installation status output.
 
-  ``EXCLUDE_EMPTY_DIRECTORIES``
-    .. versionadded:: 4.1
-
-    Exclude empty directories from installation.  A directory is
-    considered empty if it contains no files, no symbolic links,
-    and no non-empty subdirectories.
-
   ``FILES_MATCHING``
     This option may be given before the first ``<match-rule>`` to
     disable installation of files (but not directories) not matched

Help/release/4.1.rst

@@ -71,10 +71,6 @@ Commands
   ``POPULATE`` subcommands for interfacing CMake targets with pkg-config based
   dependencies.
 
-* The :command:`install(DIRECTORY)` command gained a new
-  ``EXCLUDE_EMPTY_DIRECTORIES`` option to skip installation
-  of empty directories.
-
 * The :command:`project` command now has experimental support for the
   ``COMPAT_VERSION`` keyword, gated by
   ``CMAKE_EXPERIMENTAL_EXPORT_PACKAGE_INFO``.

Source/cmFileCopier.cxx

@@ -25,10 +25,6 @@
 
 #include <cstring>
 #include <sstream>
-#include <utility>
-
-#include <cm/string_view>
-#include <cmext/string_view>
 
 using namespace cmFSPermissions;
 
@@ -298,13 +294,6 @@ bool cmFileCopier::CheckKeyword(std::string const& arg)
       this->Doing = DoingNone;
       this->MatchlessFiles = false;
     }
-  } else if (arg == "EXCLUDE_EMPTY_DIRECTORIES") {
-    if (this->CurrentMatchRule) {
-      this->NotAfterMatch(arg);
-    } else {
-      this->Doing = DoingNone;
-      this->ExcludeEmptyDirectories = true;
-    }
   } else {
     return false;
   }
@@ -658,29 +647,6 @@ bool cmFileCopier::InstallFile(std::string const& fromFile,
   return this->SetPermissions(toFile, permissions);
 }
 
-static bool IsEmptyDirectory(std::string const& path,
-                             std::unordered_map<std::string, bool>& cache)
-{
-  auto i = cache.find(path);
-  if (i == cache.end()) {
-    bool isEmpty = (!cmSystemTools::FileIsSymlink(path) &&
-                    cmSystemTools::FileIsDirectory(path));
-    if (isEmpty) {
-      cmsys::Directory d;
-      d.Load(path);
-      unsigned long numFiles = d.GetNumberOfFiles();
-      for (unsigned long fi = 0; isEmpty && fi < numFiles; ++fi) {
-        std::string const& name = d.GetFileName(fi);
-        if (name != "."_s && name != ".."_s) {
-          isEmpty = IsEmptyDirectory(d.GetFilePath(fi), cache);
-        }
-      }
-    }
-    i = cache.emplace(path, isEmpty).first;
-  }
-  return i->second;
-}
-
 bool cmFileCopier::InstallDirectory(std::string const& source,
                                     std::string const& destination,
                                     MatchProperties match_properties)
@@ -753,11 +719,6 @@ bool cmFileCopier::InstallDirectory(std::string const& source,
           strcmp(dir.GetFile(fileNum), "..") == 0)) {
       std::string fromPath = cmStrCat(source, '/', dir.GetFile(fileNum));
       std::string toPath = cmStrCat(destination, '/', dir.GetFile(fileNum));
-      if (this->ExcludeEmptyDirectories &&
-          IsEmptyDirectory(fromPath, this->DirEmptyCache)) {
-        continue;
-      }
-
       if (!this->Install(fromPath, toPath)) {
         return false;
       }

Source/cmFileCopier.h

@@ -5,7 +5,6 @@
 #include "cmConfigure.h" // IWYU pragma: keep
 
 #include <string>
-#include <unordered_map>
 #include <vector>
 
 #include "cmsys/RegularExpression.hxx"
@@ -31,7 +30,6 @@ struct cmFileCopier
   char const* Name;
   bool Always = false;
   cmFileTimeCache FileTimes;
-  std::unordered_map<std::string, bool> DirEmptyCache;
 
   // Whether to install a file not matching any expression.
   bool MatchlessFiles = true;
@@ -91,7 +89,6 @@ struct cmFileCopier
   bool UseGivenPermissionsFile = false;
   bool UseGivenPermissionsDir = false;
   bool UseSourcePermissions = true;
-  bool ExcludeEmptyDirectories = false;
   bool FollowSymlinkChain = false;
   std::string Destination;
   std::string FilesFromDir;

Source/cmInstallCommand.cxx

@@ -1787,14 +1787,6 @@ bool HandleDirectoryMode(std::vector<std::string> const& args,
       }
       exclude_from_all = true;
       doing = DoingNone;
-    } else if (args[i] == "EXCLUDE_EMPTY_DIRECTORIES") {
-      if (in_match_mode) {
-        status.SetError(cmStrCat(args[0], " does not allow \"", args[i],
-                                 "\" after PATTERN or REGEX."));
-        return false;
-      }
-      literal_args += " EXCLUDE_EMPTY_DIRECTORIES";
-      doing = DoingNone;
     } else if (doing == DoingDirs) {
       // Convert this directory to a full path.
       std::string dir = args[i];