From 828f78ccd3cccb0eebe1da183a543cf3c716ba6c Mon Sep 17 00:00:00 2001
From: Jarek Kobus <jaroslaw.kobus@qt.io>
Date: Wed, 13 Dec 2023 17:48:48 +0100
Subject: [PATCH] KillAppStep: Always finish the step with success

Fixes: QTCREATORBUG-30024
Change-Id: I7f2f3167bd3620a5a313fe6359c4ee58ff0d9e03
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
---
 src/plugins/remotelinux/killappstep.cpp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/plugins/remotelinux/killappstep.cpp b/src/plugins/remotelinux/killappstep.cpp
index b70e597fb84..80c86584200 100644
--- a/src/plugins/remotelinux/killappstep.cpp
+++ b/src/plugins/remotelinux/killappstep.cpp
@@ -58,7 +58,11 @@ GroupItem KillAppStep::deployRecipe()
         addProgressMessage(Tr::tr("Failed to kill remote application. "
                                     "Assuming it was not running."));
     };
-    return DeviceProcessKillerTask(setupHandler, doneHandler, errorHandler);
+    const Group root {
+        finishAllAndDone,
+        DeviceProcessKillerTask(setupHandler, doneHandler, errorHandler)
+    };
+    return root;
 }
 
 KillAppStepFactory::KillAppStepFactory()