mirror/CMake
1
0
Fork 0
mirror of https://github.com/Kitware/CMake.git synced 2025-12-30 18:29:37 -06:00
Code Issues Packages Projects Releases Wiki Activity

Windows: Revert "Prefer junctions for directory symlinks"

Browse Source 
Revert commit 5101d586c4 (Windows: Prefer junctions for directory
symlinks, 2022-02-22, v3.24.0-rc1~575^2).  Junctions do not support
`../` and other non-canonical paths.  Revert their use pending further
investigation.

Fixes: #23781
Issue: #23257
This commit is contained in:
Brad King
2022-08-01 10:59:07 -04:00
parent d5977ee359
commit a4f1c0fac8
1 changed files with 2 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
Source/cmSystemTools.cxx
View File

@@ -3358,22 +3358,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)