]> granicus.if.org Git - llvm/commitdiff
Support: Work around missing SetFileInformationByHandle on Wine
authorHans Wennborg <hans@hanshq.net>
Wed, 11 Oct 2017 22:04:14 +0000 (22:04 +0000)
committerHans Wennborg <hans@hanshq.net>
Wed, 11 Oct 2017 22:04:14 +0000 (22:04 +0000)
In r315079, fs::rename was reimplemented in terms of CreateFile and
SetFileInformationByHandle. Unfortunately, the latter isn't supported by
Wine. This adds a fallback to MoveFileEx for that case.

Differential Revision: https://reviews.llvm.org/D38817

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@315520 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Support/Windows/Path.inc

index 94cb2e422c5913dbf432de429a47bc071f451360..5866f9a403a80679fb71467948c37d09919f5768 100644 (file)
@@ -413,6 +413,17 @@ std::error_code rename(const Twine &From, const Twine &To) {
   // a true error, so stop trying.
   for (unsigned Retry = 0; Retry != 200; ++Retry) {
     auto EC = rename_internal(FromHandle, To, true);
+
+    if (EC ==
+        std::error_code(ERROR_CALL_NOT_IMPLEMENTED, std::system_category())) {
+      // Wine doesn't support SetFileInformationByHandle in rename_internal.
+      // Fall back to MoveFileEx.
+      if (::MoveFileExW(WideFrom.begin(), WideTo.begin(),
+                        MOVEFILE_REPLACE_EXISTING))
+        return std::error_code();
+      return mapWindowsError(GetLastError());
+    }
+
     if (!EC || EC != errc::permission_denied)
       return EC;