Use copy instead of rename for moving extracted archive directories

src/AppInstallerCLICore/PortableInstaller.cpp

@@ -107,8 +107,17 @@ namespace AppInstaller::CLI::Portable
         }
         else if (fileType == PortableFileType::Directory)
         {
-            AICLI_LOG(Core, Info, << "Moving directory to: " << filePath);
-            Filesystem::RenameFile(entry.CurrentPath, filePath);
+            if (Filesystem::IsSameVolume(entry.CurrentPath, filePath))
+            {
+                AICLI_LOG(Core, Info, << "Renaming directory to: " << filePath);
+                Filesystem::RenameFile(entry.CurrentPath, filePath);
+            }
+            else
+            {
+                // Copy directory instead of renaming as there is a known issue with renaming across drives.
+                AICLI_LOG(Core, Info, << "Copying directory to: " << filePath);
+                std::filesystem::copy(entry.CurrentPath, filePath, std::filesystem::copy_options::overwrite_existing | std::filesystem::copy_options::recursive);
+            }
         }
         else if (entry.FileType == PortableFileType::Symlink)
         {

src/AppInstallerCLITests/Filesystem.cpp

@@ -56,4 +56,30 @@ TEST_CASE("VerifySymlink", "[filesystem]")
     std::filesystem::remove(symlinkPath);
 
     REQUIRE_FALSE(SymlinkExists(symlinkPath));
+}
+
+TEST_CASE("VerifyIsSameVolume", "[filesystem]")
+{
+    // Note: Pipeline build machine uses 'D:\' as the volume.
+    std::filesystem::path path1 = L"C:\\Program Files\\WinGet\\Packages";
+    std::filesystem::path path2 = L"c:\\Users\\testUser\\AppData\\Local\\Microsoft\\WinGet\\Packages";
+    std::filesystem::path path3 = L"localPath\\test\\folder";
+    std::filesystem::path path4 = L"\\test\\folder";
+
+    std::filesystem::path path5 = L"D:\\test\\folder";
+    std::filesystem::path path6 = L"F:\\test\\folder";
+    std::filesystem::path path7 = L"d:\\randomFolder";
+
+    // Verify that a relative path points to the current volume.
+    REQUIRE(IsSameVolume(path1, path2));
+    REQUIRE(IsSameVolume(path5, path7));
+    REQUIRE(IsSameVolume(path3, path4));
+
+    REQUIRE_FALSE(IsSameVolume(path1, path5));
+    REQUIRE_FALSE(IsSameVolume(path1, path6));
+    REQUIRE_FALSE(IsSameVolume(path2, path5));
+    REQUIRE_FALSE(IsSameVolume(path2, path6));
+    REQUIRE_FALSE(IsSameVolume(path3, path6));
+    REQUIRE_FALSE(IsSameVolume(path5, path6));
+    REQUIRE_FALSE(IsSameVolume(path4, path6));
 }

src/AppInstallerCommonCore/Filesystem.cpp

@@ -238,4 +238,18 @@ namespace AppInstaller::Filesystem
         THROW_IF_FAILED(SHGetKnownFolderPath(id, KF_FLAG_NO_ALIAS | KF_FLAG_DONT_VERIFY | KF_FLAG_NO_PACKAGE_REDIRECTION, NULL, &knownFolder));
         return knownFolder.get();
     }
+
+    bool IsSameVolume(const std::filesystem::path& path1, const std::filesystem::path& path2)
+    {
+        std::wstring path1Volume;
+        std::wstring path2Volume;
+        DWORD path1Length = static_cast<DWORD>(path1.wstring().length() + 1);
+        DWORD path2Length = static_cast<DWORD>(path2.wstring().length() + 1);
+        path1Volume.resize(path1Length);
+        path2Volume.resize(path2Length);
+
+        GetVolumePathNameW(path1.c_str(), path1Volume.data(), path1Length);
+        GetVolumePathNameW(path2.c_str(), path2Volume.data(), path2Length);
+        return Utility::CaseInsensitiveEquals(Utility::ConvertToUTF8(path1Volume.data()), Utility::ConvertToUTF8(path2Volume.data()));
+    }
 }

src/AppInstallerCommonCore/Public/winget/Filesystem.h

@@ -42,4 +42,7 @@ namespace AppInstaller::Filesystem
 
     // Gets the path of a known folder.
     std::filesystem::path GetKnownFolderPath(const KNOWNFOLDERID& id);
+
+    // Verifies that the paths are on the same volume.
+    bool IsSameVolume(const std::filesystem::path& path1, const std::filesystem::path& path2);
 }