From 8662d0a9c036f5ca8128f048421bd9be6249a1b8 Mon Sep 17 00:00:00 2001
From: Christian Kandeler <christian.kandeler@qt.io>
Date: Fri, 21 Dec 2018 10:47:39 +0100
Subject: [PATCH] RemoteLinux: Make it clear that rsync is not strictly needed

... by adding some soothing output to the device tester.

Fixes: QTCREATORBUG-21749
Change-Id: I6d2125d50716192a7fcd09f41269a32b41ad9be6
Reviewed-by: James McDonnell <jmcdonnell@blackberry.com>
---
 src/plugins/remotelinux/linuxdevicetester.cpp | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/plugins/remotelinux/linuxdevicetester.cpp b/src/plugins/remotelinux/linuxdevicetester.cpp
index c89f41232a0..a555c22e861 100644
--- a/src/plugins/remotelinux/linuxdevicetester.cpp
+++ b/src/plugins/remotelinux/linuxdevicetester.cpp
@@ -251,7 +251,10 @@ void GenericLinuxDeviceTester::handleRsyncFinished()
     TestResult result = TestSuccess;
     if (!error.isEmpty()) {
         emit errorMessage(error);
-        if (!d->sftpWorks) {
+        if (d->sftpWorks) {
+            emit progressMessage(tr("SFTP will be used for deployment, because rsync "
+                                    "is not available.\n"));
+        } else {
             emit errorMessage(tr("Deployment to this device will not work out of the box.\n"));
             result = TestFailure;
         }