From 9cc5c072990e90777fc8df69c26a92975e788054 Mon Sep 17 00:00:00 2001
From: AtlasLinden <114031241+AtlasLinden@users.noreply.github.com>
Date: Thu, 10 Jul 2025 10:36:16 -0700
Subject: New method to handle removing previous viewer

Moving previous viewer to trash instead of "removing" it.
---
 .github/workflows/qatest.yaml | 21 ++++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

(limited to '.github/workflows')

diff --git a/.github/workflows/qatest.yaml b/.github/workflows/qatest.yaml
index 29d55c775f..4e10900441 100644
--- a/.github/workflows/qatest.yaml
+++ b/.github/workflows/qatest.yaml
@@ -514,13 +514,28 @@ jobs:
           APP_NAME=$(basename "$APP_PATH")
           DEST_PATH="/Applications/$APP_NAME"
 
-          # Remove existing installation if it exists to avoid permission conflicts
+          # Handle existing installation
           if [ -d "$DEST_PATH" ]; then
-            echo "Removing existing installation..."
-            sudo rm -rf "$DEST_PATH"
+            echo "Found existing installation at: $DEST_PATH"
+            echo "Moving existing installation to trash..."
+
+            # Move to trash instead of force removing
+            TRASH_PATH="$HOME/.Trash/$(date +%Y%m%d_%H%M%S)_$APP_NAME"
+            mv "$DEST_PATH" "$TRASH_PATH" || {
+              echo "⚠️ Could not move to trash, trying direct removal..."
+              rm -rf "$DEST_PATH" || {
+                echo "❌ Could not remove existing installation"
+                echo "Please manually remove: $DEST_PATH"
+                exit 1
+              }
+            }
+
+            echo "✅ Existing installation handled successfully"
           fi
 
           # Copy the .app to /Applications
+          echo "Copying app from: $APP_PATH"
+          echo "To destination: /Applications/"
           cp -R "$APP_PATH" /Applications/
 
           # Verify the app was copied successfully
-- 
cgit v1.3