ProjectExplorer: Replace RunConfiguration::executable

... by a RunConfiguration::commandLine().

That's what is typically consumed, and removes the need for some
custom runnable() implementations.

Change-Id: I7700b12cdd0965802a0add977b432314734a5a72
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>

1 files changed, 7 insertions, 15 deletions

diff --git a/src/plugins/pythoneditor/pythoneditorplugin.cpp b/src/plugins/pythoneditor/pythoneditorplugin.cpp
index 9a494b47b0..8dd87ebbda 100644
--- a/src/plugins/pythoneditor/pythoneditorplugin.cpp
+++ b/src/plugins/pythoneditor/pythoneditorplugin.cpp
@@ -246,7 +246,6 @@ public:
 
 private:
     void doAdditionalSetup(const RunConfigurationCreationInfo &) final { updateTargetInformation(); }
-    Runnable runnable() const final;
 
     bool supportsDebugger() const { return true; }
     QString mainScript() const { return aspect<MainScriptAspect>()->value(); }
@@ -275,12 +274,16 @@ PythonRunConfiguration::PythonRunConfiguration(Target *target, Core::Id id)
     scriptAspect->setDisplayStyle(BaseStringAspect::LabelDisplay);
 
     addAspect<LocalEnvironmentAspect>(target);
-    addAspect<ArgumentsAspect>();
+
+    auto argumentsAspect = addAspect<ArgumentsAspect>();
+
     addAspect<TerminalAspect>();
 
     setOutputFormatter<PythonOutputFormatter>();
-    setExecutableGetter([this] {
-        return FilePath::fromString(aspect<InterpreterAspect>()->value());
+    setCommandLineGetter([this, interpreterAspect, argumentsAspect] {
+        CommandLine cmd{FilePath::fromString(interpreterAspect->value()), {mainScript()}};
+        cmd.addArgs(argumentsAspect->arguments(macroExpander()), CommandLine::Raw);
+        return cmd;
     });
 
     connect(target, &Target::applicationTargetsChanged,
@@ -297,17 +300,6 @@ void PythonRunConfiguration::updateTargetInformation()
     aspect<MainScriptAspect>()->setValue(script);
 }
 
-Runnable PythonRunConfiguration::runnable() const
-{
-    CommandLine cmd{executable(), {mainScript()}};
-    cmd.addArgs(aspect<ArgumentsAspect>()->arguments(macroExpander()), CommandLine::Raw);
-
-    Runnable r;
-    r.setCommandLine(cmd);
-    r.environment = aspect<EnvironmentAspect>()->environment();
-    return r;
-}
-
 class PythonRunConfigurationFactory : public RunConfigurationFactory
 {
 public: