Kitware/CMake
1
0
Fork 0
mirror of https://github.com/Kitware/CMake.git synced 2025-10-19 19:43:23 +08:00
Code Issues Projects Releases Wiki Activity

cmSystemTools: Factor out MoveFileExW call in RenameFile

Browse Source
This commit is contained in:
Brad King
2020-08-27 15:43:51 -04:00
parent 35039286eb
commit 97fc44f70e
1 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
Source/cmSystemTools.cxx
View File

@@ -881,6 +881,16 @@ void cmSystemTools::InitializeLibUV()
#endif
}
#ifdef _WIN32
namespace {
bool cmMoveFile(std::wstring const& oldname, std::wstring const& newname)
{
return MoveFileExW(oldname.c_str(), newname.c_str(),
MOVEFILE_REPLACE_EXISTING);
}
}
#endif
bool cmSystemTools::RenameFile(const std::string& oldname,
const std::string& newname)
{
@@ -893,11 +903,9 @@ bool cmSystemTools::RenameFile(const std::string& oldname,
Try multiple times since we may be racing against another process
creating/opening the destination file just before our MoveFileEx. */
WindowsFileRetry retry = cmSystemTools::GetWindowsFileRetry();
while (
!MoveFileExW(SystemTools::ConvertToWindowsExtendedPath(oldname).c_str(),
SystemTools::ConvertToWindowsExtendedPath(newname).c_str(),
MOVEFILE_REPLACE_EXISTING) &&
--retry.Count) {
while (!cmMoveFile(SystemTools::ConvertToWindowsExtendedPath(oldname),
SystemTools::ConvertToWindowsExtendedPath(newname)) &&
--retry.Count) {
DWORD last_error = GetLastError();
// Try again only if failure was due to access/sharing permissions.
if (last_error != ERROR_ACCESS_DENIED &&