ProjectExplorer: Reduce use of Runnable in SimpleTargetRunner

Runnable functionality is nowadays mostly accessed more directly
in QtcProcess and its setup functions.

Change-Id: I2a2b5433aef1d464dc58d5a35069376dee051d57
Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>

src/plugins/baremetal/debugservers/gdb/gdbserverprovider.cpp

@@ -199,11 +199,9 @@ RunWorker *GdbServerProvider::targetRunner(RunControl *runControl) const
     if (m_startupMode != GdbServerProvider::StartupOnNetwork)
         return nullptr;
 
-    Runnable r;
-    r.command = command();
     // Command arguments are in host OS style as the bare metal's GDB servers are launched
     // on the host, not on that target.
-    return new GdbServerProviderRunner(runControl, r);
+    return new GdbServerProviderRunner(runControl, command());
 }
 
 bool GdbServerProvider::fromMap(const QVariantMap &data)
@@ -341,14 +339,15 @@ QString GdbServerProviderConfigWidget::defaultResetCommandsTooltip()
 // GdbServerProviderRunner
 
 GdbServerProviderRunner::GdbServerProviderRunner(ProjectExplorer::RunControl *runControl,
-                                                 const ProjectExplorer::Runnable &runnable)
+                                                 const Utils::CommandLine &commandLine)
     : SimpleTargetRunner(runControl)
 {
     setId("BareMetalGdbServer");
     // Baremetal's GDB servers are launched on the host, not on the target.
-    Runnable devicelessRunnable = runnable;
+    setStartModifier([this, commandLine] {
-    devicelessRunnable.device.reset();
+        setCommandLine(commandLine);
-    setStarter([this, devicelessRunnable] { doStart(devicelessRunnable); });
+        forceRunOnHost();
+    });
 }
 
 } // namespace Internal

src/plugins/baremetal/debugservers/gdb/gdbserverprovider.h

@@ -126,7 +126,7 @@ class GdbServerProviderRunner final : public ProjectExplorer::SimpleTargetRunner
 {
 public:
     explicit GdbServerProviderRunner(ProjectExplorer::RunControl *runControl,
-                                     const ProjectExplorer::Runnable &runnable);
+                                     const Utils::CommandLine &commandLine);
 };
 
 } // namespace Internal

src/plugins/boot2qt/qdbplugin.cpp

@@ -142,13 +142,11 @@ public:
     QdbDeviceRunSupport(RunControl *runControl)
         : SimpleTargetRunner(runControl)
     {
-        setStarter([this, runControl] {
+        setStartModifier([this] {
-            Runnable r = runControl->runnable();
+            CommandLine cmd;
-            // FIXME: Spaces!
+            cmd.setExecutable(FilePath::fromString(Constants::AppcontrollerFilepath));
-            r.command.setArguments(r.command.executable().toString() + ' ' + r.command.arguments());
+            cmd.addCommandLineAsArgs(commandLine());
-            r.command.setExecutable(FilePath::fromString(Constants::AppcontrollerFilepath));
+            setCommandLine(cmd);
-            r.device = runControl->device();
-            doStart(r);
         });
     }
 };

src/plugins/debugger/debuggerruncontrol.cpp

@@ -1042,35 +1042,31 @@ DebugServerRunner::DebugServerRunner(RunControl *runControl, DebugServerPortsGat
    : SimpleTargetRunner(runControl)
 {
     setId("DebugServerRunner");
-    const Runnable mainRunnable = runControl->runnable();
     addStartDependency(portsGatherer);
 
     QTC_ASSERT(portsGatherer, reportFailure(); return);
 
-    setStarter([this, runControl, mainRunnable, portsGatherer] {
+    setStartModifier([this, runControl, portsGatherer] {
         QTC_ASSERT(portsGatherer, reportFailure(); return);
 
-        Runnable debugServer;
-        debugServer.environment = mainRunnable.environment;
-        debugServer.workingDirectory = mainRunnable.workingDirectory;
-
-        QStringList args = ProcessArgs::splitArgs(mainRunnable.command.arguments(), OsTypeLinux);
-
         const bool isQmlDebugging = portsGatherer->useQmlServer();
         const bool isCppDebugging = portsGatherer->useGdbServer();
 
+        CommandLine cmd;
+
+        QStringList args = ProcessArgs::splitArgs(commandLine().arguments(), OsTypeLinux);
         if (isQmlDebugging) {
             args.prepend(QmlDebug::qmlDebugTcpArguments(QmlDebug::QmlDebuggerServices,
                                                         portsGatherer->qmlServer()));
         }
         if (isQmlDebugging && !isCppDebugging) {
-            debugServer.command.setExecutable(mainRunnable.command.executable()); // FIXME: Case should not happen?
+            cmd.setExecutable(commandLine().executable()); // FIXME: Case should not happen?
         } else {
-            debugServer.command.setExecutable(runControl->device()->debugServerPath());
+            cmd.setExecutable(runControl->device()->debugServerPath());
-            if (debugServer.command.isEmpty())
+            if (cmd.isEmpty())
-                debugServer.command.setExecutable("gdbserver");
+                cmd.setExecutable("gdbserver");
             args.clear();
-            if (debugServer.command.executable().toString().contains("lldb-server")) {
+            if (cmd.executable().toString().contains("lldb-server")) {
                 args.append("platform");
                 args.append("--listen");
                 args.append(QString("*:%1").arg(portsGatherer->gdbServer().port()));
@@ -1086,9 +1082,9 @@ DebugServerRunner::DebugServerRunner(RunControl *runControl, DebugServerPortsGat
                     args.append(QString::number(m_pid.pid()));
             }
         }
-        debugServer.command.setArguments(ProcessArgs::joinArgs(args, OsTypeLinux));
+        cmd.setArguments(ProcessArgs::joinArgs(args, OsTypeLinux));
-        debugServer.device = runControl->device();
+
-        doStart(debugServer);
+        setCommandLine(cmd);
     });
 }
 

src/plugins/mcusupport/mcusupportrunconfiguration.cpp

@@ -88,15 +88,12 @@ public:
     FlashAndRunWorker(RunControl *runControl)
         : SimpleTargetRunner(runControl)
     {
-        setStarter([this, runControl] {
+        setStartModifier([this, runControl] {
             const Target *target = runControl->target();
-            Runnable r;
+            setCommandLine({cmakeFilePath(target), runControl->aspect<StringAspect>()->value,
-            r.command = {cmakeFilePath(target),
+                            CommandLine::Raw});
-                         runControl->aspect<StringAspect>()->value,
+            setWorkingDirectory(target->activeBuildConfiguration()->buildDirectory());
-                         CommandLine::Raw};
+            setEnvironment(target->activeBuildConfiguration()->environment());
-            r.workingDirectory = target->activeBuildConfiguration()->buildDirectory();
-            r.environment = target->activeBuildConfiguration()->environment();
-            SimpleTargetRunner::doStart(r);
         });
     }
 };

src/plugins/projectexplorer/runcontrol.cpp

@@ -1246,7 +1246,7 @@ public:
 
     ProcessResultData m_resultData;
 
-    std::function<void()> m_starter;
+    std::function<void()> m_startModifier;
 
     bool m_stopReported = false;
     bool m_stopForced = false;
@@ -1481,17 +1481,6 @@ SimpleTargetRunner::SimpleTargetRunner(RunControl *runControl)
 
 SimpleTargetRunner::~SimpleTargetRunner() = default;
 
-void SimpleTargetRunner::start()
-{
-    if (d->m_starter) {
-        d->m_starter();
-    } else {
-        Runnable runnable = runControl()->runnable();
-        runnable.device = runControl()->device();
-        doStart(runnable);
-    }
-}
-
 void SimpleTargetRunnerPrivate::forwardDone()
 {
     if (m_stopReported)
@@ -1523,8 +1512,14 @@ void SimpleTargetRunnerPrivate::forwardStarted()
     q->reportStarted();
 }
 
-void SimpleTargetRunner::doStart(const Runnable &runnable)
+void SimpleTargetRunner::start()
 {
+    d->m_runnable = runControl()->runnable();
+    d->m_runnable.device = runControl()->device();
+
+    if (d->m_startModifier)
+        d->m_startModifier();
+
     bool useTerminal = false;
     if (auto terminalAspect = runControl()->aspect<TerminalAspect>())
         useTerminal = terminalAspect->useTerminal;
@@ -1539,16 +1534,15 @@ void SimpleTargetRunner::doStart(const Runnable &runnable)
     d->m_process.setTerminalMode(useTerminal ? Utils::TerminalMode::On : Utils::TerminalMode::Off);
     d->m_runAsRoot = runAsRoot;
 
-    const QString msg = RunControl::tr("Starting %1...").arg(runnable.command.toUserOutput());
+    const QString msg = RunControl::tr("Starting %1...").arg(d->m_runnable.command.toUserOutput());
     appendMessage(msg, Utils::NormalMessageFormat);
 
-    const bool isDesktop = runnable.device.isNull()
+    const bool isDesktop = d->m_runnable.device.isNull()
-                        || runnable.device.dynamicCast<const DesktopDevice>();
+                        || d->m_runnable.device.dynamicCast<const DesktopDevice>();
-    if (isDesktop && runnable.command.isEmpty()) {
+    if (isDesktop && d->m_runnable.command.isEmpty()) {
         reportFailure(RunControl::tr("No executable specified."));
         return;
     }
-    d->m_runnable = runnable;
     d->start();
 }
 
@@ -1558,11 +1552,40 @@ void SimpleTargetRunner::stop()
     d->stop();
 }
 
-void SimpleTargetRunner::setStarter(const std::function<void ()> &starter)
+void SimpleTargetRunner::setStartModifier(const std::function<void ()> &startModifier)
 {
-    d->m_starter = starter;
+    d->m_startModifier = startModifier;
 }
 
+CommandLine SimpleTargetRunner::commandLine() const
+{
+    return d->m_runnable.command;
+}
+
+void SimpleTargetRunner::setCommandLine(const Utils::CommandLine &commandLine)
+{
+    d->m_runnable.command = commandLine;
+}
+
+void SimpleTargetRunner::setEnvironment(const Environment &environment)
+{
+    d->m_runnable.environment = environment;
+}
+
+void SimpleTargetRunner::setWorkingDirectory(const FilePath &workingDirectory)
+{
+    d->m_runnable.workingDirectory = workingDirectory;
+}
+
+void SimpleTargetRunner::forceRunOnHost()
+{
+    d->m_runnable.device = {};
+    const FilePath executable = d->m_runnable.command.executable();
+    if (executable.needsDevice()) {
+        QTC_CHECK(false);
+        d->m_runnable.command.setExecutable(FilePath::fromString(executable.path()));
+    }
+}
 
 // RunWorkerPrivate
 

src/plugins/projectexplorer/runcontrol.h

@@ -298,14 +298,22 @@ public:
     ~SimpleTargetRunner() override;
 
 protected:
-    void setStarter(const std::function<void()> &starter);
+    void setStartModifier(const std::function<void()> &startModifier);
-    void doStart(const Runnable &runnable);
+
+    Utils::CommandLine commandLine() const;
+    void setCommandLine(const Utils::CommandLine &commandLine);
+
+    void setEnvironment(const Utils::Environment &environment);
+    void setWorkingDirectory(const Utils::FilePath &workingDirectory);
+
+    void forceRunOnHost();
 
 private:
     void start() final;
     void stop() final;
 
     const Runnable &runnable() const = delete;
+    void setRunnable(const Runnable &) = delete;
 
     const std::unique_ptr<Internal::SimpleTargetRunnerPrivate> d;
 };

src/plugins/qmlpreview/qmlpreviewruncontrol.cpp

@@ -41,6 +41,8 @@
 #include <utils/url.h>
 #include <utils/fileutils.h>
 
+using namespace Utils;
+
 namespace QmlPreview {
 
 static const QString QmlServerUrl = "QmlServerUrl";
@@ -126,9 +128,10 @@ LocalQmlPreviewSupport::LocalQmlPreviewSupport(ProjectExplorer::RunControl *runC
     addStopDependency(preview);
     addStartDependency(preview);
 
-    setStarter([this, runControl, serverUrl] {
+    setStartModifier([this, runControl, serverUrl] {
-        ProjectExplorer::Runnable runnable = runControl->runnable();
+        CommandLine cmd = commandLine();
-        QStringList qmlProjectRunConfigurationArguments = runnable.command.splitArguments();
+
+        QStringList qmlProjectRunConfigurationArguments = cmd.splitArguments();
 
         const auto currentTarget = runControl->target();
         const auto *qmlBuildSystem = qobject_cast<QmlProjectManager::QmlBuildSystem *>(currentTarget->buildSystem());
@@ -142,15 +145,15 @@ LocalQmlPreviewSupport::LocalQmlPreviewSupport(ProjectExplorer::RunControl *runC
 
             if (!currentFile.isEmpty() && qmlProjectRunConfigurationArguments.last().contains(mainScriptFromProject)) {
                 qmlProjectRunConfigurationArguments.removeLast();
-                runnable.command = Utils::CommandLine(runnable.command.executable(), qmlProjectRunConfigurationArguments);
+                cmd = Utils::CommandLine(cmd.executable(), qmlProjectRunConfigurationArguments);
-                runnable.command.addArg(currentFile);
+                cmd.addArg(currentFile);
             }
         }
 
-        runnable.command.addArg(QmlDebug::qmlDebugLocalArguments(QmlDebug::QmlPreviewServices,
+        cmd.addArg(QmlDebug::qmlDebugLocalArguments(QmlDebug::QmlPreviewServices, serverUrl.path()));
-                                                                 serverUrl.path()));
+        setCommandLine(cmd);
-        runnable.device.reset();
+
-        doStart(runnable);
+        forceRunOnHost();
     });
 }
 

src/plugins/qmlprofiler/qmlprofilerruncontrol.cpp

@@ -25,7 +25,6 @@
 
 #include "qmlprofilerruncontrol.h"
 
-#include "qmlprofilerclientmanager.h"
 #include "qmlprofilertool.h"
 
 #include <coreplugin/icore.h>
@@ -236,8 +235,7 @@ LocalQmlProfilerSupport::LocalQmlProfilerSupport(RunControl *runControl, const Q
     // In the TCP case, it doesn't hurt either to start the profiler before.
     addStartDependency(profiler);
 
-    setStarter([this, runControl, profiler, serverUrl] {
+    setStartModifier([this, profiler, serverUrl] {
-        Runnable debuggee = runControl->runnable();
 
         QUrl serverUrl = profiler->serverUrl();
         QString code;
@@ -251,12 +249,13 @@ LocalQmlProfilerSupport::LocalQmlProfilerSupport(RunControl *runControl, const Q
         QString arguments = Utils::ProcessArgs::quoteArg(
                                 QmlDebug::qmlDebugCommandLineArguments(QmlDebug::QmlProfilerServices, code, true));
 
-        if (!debuggee.command.arguments().isEmpty())
+        Utils::CommandLine cmd = commandLine();
-            arguments += ' ' + debuggee.command.arguments();
+        const QString oldArgs = cmd.arguments();
+        cmd.setArguments(arguments);
+        cmd.addArgs(oldArgs, Utils::CommandLine::Raw);
+        setCommandLine(cmd);
 
-        debuggee.command.setArguments(arguments);
+        forceRunOnHost();
-        debuggee.device.reset();
-        doStart(debuggee);
     });
 }
 

src/plugins/qnx/qnxanalyzesupport.cpp

@@ -56,14 +56,13 @@ QnxQmlProfilerSupport::QnxQmlProfilerSupport(RunControl *runControl)
     profiler->addStartDependency(this);
     addStopDependency(profiler);
 
-    setStarter([this, runControl, portsGatherer, profiler] {
+    setStartModifier([this, portsGatherer, profiler] {
         const QUrl serverUrl = portsGatherer->findEndPoint();
         profiler->recordData("QmlServerUrl", serverUrl);
 
-        Runnable r = runControl->runnable();
+        CommandLine cmd = commandLine();
-        r.command.addArg(QmlDebug::qmlDebugTcpArguments(QmlDebug::QmlProfilerServices, serverUrl));
+        cmd.addArg(QmlDebug::qmlDebugTcpArguments(QmlDebug::QmlProfilerServices, serverUrl));
-        r.device = runControl->device();
+        setCommandLine(cmd);
-        doStart(r);
     });
 }
 

src/plugins/qnx/qnxdebugsupport.cpp

@@ -26,11 +26,9 @@
 #include "qnxdebugsupport.h"
 
 #include "qnxconstants.h"
-#include "qnxdevice.h"
 #include "qnxrunconfiguration.h"
 #include "slog2inforunner.h"
 #include "qnxqtversion.h"
-#include "qnxutils.h"
 
 #include <coreplugin/icore.h>
 
@@ -103,21 +101,21 @@ public:
     {
         setId("QnxDebuggeeRunner");
 
-        setStarter([this, runControl, portsGatherer] {
+        setStartModifier([this, portsGatherer] {
-            Runnable r = runControl->runnable();
+            CommandLine cmd = commandLine();
             QStringList arguments;
             if (portsGatherer->useGdbServer()) {
                 int pdebugPort = portsGatherer->gdbServer().port();
-                r.command.setExecutable(FilePath::fromString(QNX_DEBUG_EXECUTABLE));
+                cmd.setExecutable(FilePath::fromString(QNX_DEBUG_EXECUTABLE));
                 arguments.append(QString::number(pdebugPort));
             }
             if (portsGatherer->useQmlServer()) {
                 arguments.append(QmlDebug::qmlDebugTcpArguments(QmlDebug::QmlDebuggerServices,
                                                                 portsGatherer->qmlServer()));
             }
-            r.command.setArguments(ProcessArgs::joinArgs(arguments));
+            cmd.setArguments(ProcessArgs::joinArgs(arguments));
-            r.device = runControl->device();
+            setCommandLine(cmd);
-            doStart(r);
+
         });
     }
 };
@@ -197,13 +195,9 @@ public:
         setId("PDebugRunner");
         addStartDependency(portsGatherer);
 
-        setStarter([this, runControl, portsGatherer] {
+        setStartModifier([this, portsGatherer] {
             const int pdebugPort = portsGatherer->gdbServer().port();
-
+            setCommandLine({QNX_DEBUG_EXECUTABLE, {QString::number(pdebugPort)}});
-            Runnable r;
-            r.command = {QNX_DEBUG_EXECUTABLE, {QString::number(pdebugPort)}};
-            r.device = runControl->device();
-            doStart(r);
         });
     }
 };

src/plugins/remotelinux/remotelinuxqmltoolingsupport.cpp

@@ -51,16 +51,15 @@ RemoteLinuxQmlToolingSupport::RemoteLinuxQmlToolingSupport(RunControl *runContro
     runworker->addStartDependency(this);
     addStopDependency(runworker);
 
-    setStarter([this, runControl, portsGatherer, runworker] {
+    setStartModifier([this, runControl, portsGatherer, runworker] {
         const QUrl serverUrl = portsGatherer->findEndPoint();
         runworker->recordData("QmlServerUrl", serverUrl);
 
         QmlDebug::QmlDebugServicesPreset services = QmlDebug::servicesForRunMode(runControl->runMode());
 
-        Runnable r = runControl->runnable();
+        CommandLine cmd = commandLine();
-        r.command.addArg(QmlDebug::qmlDebugTcpArguments(services, serverUrl));
+        cmd.addArg(QmlDebug::qmlDebugTcpArguments(services, serverUrl));
-        r.device = runControl->device();
+        setCommandLine(cmd);
-        doStart(r);
     });
 }
 

src/plugins/webassembly/webassemblyrunconfiguration.cpp

@@ -120,16 +120,14 @@ public:
         auto portsGatherer = new PortsGatherer(runControl);
         addStartDependency(portsGatherer);
 
-        setStarter([this, runControl, portsGatherer] {
+        setStartModifier([this, runControl, portsGatherer] {
-            Runnable r;
             const QString browserId =
                     runControl->aspect<WebBrowserSelectionAspect>()->currentBrowser;
-            r.command = emrunCommand(runControl->target(),
+            setCommandLine(emrunCommand(runControl->target(),
-                                     runControl->buildKey(),
+                                        runControl->buildKey(),
-                                     browserId,
+                                        browserId,
-                                     QString::number(portsGatherer->findEndPoint().port()));
+                                        QString::number(portsGatherer->findEndPoint().port())));
-            r.environment = runControl->buildEnvironment();
+            setEnvironment(runControl->buildEnvironment());
-            SimpleTargetRunner::doStart(r);
         });
     }
 };