feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

RemoteLinux: Fix potentially wrong name of deploy tool

Browse Source 
... in error messages.
For instance, one could get "rsync crashed" when it was actually sftp,
misleading the user.

Change-Id: I42bcecb8ca21c899ace78eb1397d3e71f2a0bb78
Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
This commit is contained in:
Christian Kandeler
2024-07-02 14:22:20 +02:00
parent 67d2789523
commit dfbe37542d
2 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/plugins/projectexplorer/devicesupport/filetransfer.h
View File

@@ -37,6 +37,7 @@ public:
Utils::ProcessResultData resultData() const;
static QString transferMethodName(FileTransferMethod method);
QString transferMethodName() const { return transferMethodName(transferMethod()); }
signals:
void progress(const QString &progressMessage);

12
src/plugins/remotelinux/genericdeploystep.cpp
View File

@@ -176,12 +176,16 @@ GroupItem GenericDeployStep::transferTask(const Storage<FilesToTransfer> &storag
const auto onError = [this](const FileTransfer &transfer) {
const ProcessResultData result = transfer.resultData();
if (result.m_error == QProcess::FailedToStart) {
addErrorMessage(Tr::tr("rsync failed to start: %1").arg(result.m_errorString));
addErrorMessage(Tr::tr("%1 failed to start: %2")
.arg(transfer.transferMethodName(), result.m_errorString));
} else if (result.m_exitStatus == QProcess::CrashExit) {
addErrorMessage(Tr::tr("rsync crashed."));
addErrorMessage(Tr::tr("%1 crashed.").arg(transfer.transferMethodName()));
} else if (result.m_exitCode != 0) {
addErrorMessage(Tr::tr("rsync failed with exit code %1.").arg(result.m_exitCode)
+ "\n" + result.m_errorString);
addErrorMessage(
Tr::tr("%1 failed with exit code %2.")
.arg(transfer.transferMethodName())
.arg(result.m_exitCode)
+ "\n" + result.m_errorString);
}
};
return FileTransferTask(onSetup, onError, CallDoneIf::Error);