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

AbstractRemoteLinuxDeployStep: Get rid of run preparer

Browse Source 
Make it a part of isDeploymentNecessary() instead.

Change-Id: I73bda4f75b94222b6e9475b13a373f7237969999
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
This commit is contained in:
Jarek Kobus
2023-07-07 12:16:29 +02:00
parent 243341df46
commit a119242771
4 changed files with 6 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/plugins/remotelinux/rsyncdeploystep.cpp
View File

@@ -60,13 +60,6 @@ public:
}
return isDeploymentPossible();
});
setRunPreparer([this] {
const QList<DeployableFile> files = target()->deploymentData().allFiles();
m_files.clear();
for (const DeployableFile &f : files)
m_files.append({f.localFilePath(), deviceConfiguration()->filePath(f.remoteFilePath())});
});
}
private:
@@ -84,6 +77,10 @@ private:
bool RsyncDeployStep::isDeploymentNecessary() const
{
const QList<DeployableFile> files = target()->deploymentData().allFiles();
m_files.clear();
for (const DeployableFile &f : files)
m_files.append({f.localFilePath(), deviceConfiguration()->filePath(f.remoteFilePath())});
if (ignoreMissingFiles())
Utils::erase(m_files, [](const FileToTransfer &file) { return !file.m_source.exists(); });
return !m_files.empty();