Browse Source

Merge topic 'windows-symlink-dir'

a4f1c0fac8 Windows: Revert "Prefer junctions for directory symlinks"

Acked-by: Kitware Robot <[email protected]>
Merge-request: !7530
Brad King 3 years ago
parent
commit
621ea848b0
1 changed files with 2 additions and 12 deletions
  1. 2 12
      Source/cmSystemTools.cxx

+ 2 - 12
Source/cmSystemTools.cxx

@@ -3357,22 +3357,12 @@ cmsys::Status cmSystemTools::CreateSymlink(std::string const& origName,
   uv_fs_t req;
   int flags = 0;
 #if defined(_WIN32)
-  bool const isDir = cmsys::SystemTools::FileIsDirectory(origName);
-  if (isDir) {
-    flags |= UV_FS_SYMLINK_JUNCTION;
+  if (cmsys::SystemTools::FileIsDirectory(origName)) {
+    flags |= UV_FS_SYMLINK_DIR;
   }
 #endif
   int err = uv_fs_symlink(nullptr, &req, origName.c_str(), newName.c_str(),
                           flags, nullptr);
-#if defined(_WIN32)
-  if (err && uv_fs_get_system_error(&req) == ERROR_NOT_SUPPORTED && isDir) {
-    // Try fallback to symlink for network (requires additional permissions).
-    flags ^= UV_FS_SYMLINK_JUNCTION | UV_FS_SYMLINK_DIR;
-    err = uv_fs_symlink(nullptr, &req, origName.c_str(), newName.c_str(),
-                        flags, nullptr);
-  }
-#endif
-
   cmsys::Status status;
   if (err) {
 #if defined(_WIN32)