Valgrind: Support console applications

Task-number: QTCREATORBUG-7311
Change-Id: I973136076118fd8868c6cb461ad31e107c73566e
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>

src/plugins/analyzerbase/analyzerstartparameters.h

@@ -38,6 +38,7 @@
 #include <ssh/sshconnection.h>
 #include <utils/environment.h>
 #include <projectexplorer/projectexplorerconstants.h>
+#include <projectexplorer/applicationlauncher.h>
 
 namespace Analyzer {
 
@@ -47,12 +48,15 @@ namespace Analyzer {
 class ANALYZER_EXPORT AnalyzerStartParameters
 {
 public:
-    AnalyzerStartParameters() : analyzerPort(0)
+    AnalyzerStartParameters()
+        : localRunMode(ProjectExplorer::ApplicationLauncher::Gui)
+        , analyzerPort(0)
     {}
 
     StartMode startMode;
     ProjectExplorer::RunMode runMode;
     QSsh::SshConnectionParameters connParams;
+    ProjectExplorer::ApplicationLauncher::Mode localRunMode;
 
     QString debuggee;
     QString debuggeeArgs;

src/plugins/valgrind/valgrind_test.pri

@@ -59,3 +59,5 @@ SOURCES += $$PWD/xmlprotocol/error.cpp \
     $$PWD/memcheck/memcheckrunner.cpp \
     $$PWD/valgrindrunner.cpp \
     $$PWD/valgrindprocess.cpp
+
+LIBS += -L$$IDE_PLUGIN_PATH/QtProject

src/plugins/valgrind/valgrindengine.cpp

@@ -108,6 +108,7 @@ bool ValgrindRunControl::startEngine()
     run->setEnvironment(sp.environment);
     run->setConnectionParameters(sp.connParams);
     run->setStartMode(sp.startMode);
+    run->setLocalRunMode(sp.localRunMode);
 
     connect(run, SIGNAL(processOutputReceived(QString,Utils::OutputFormat)),
             SLOT(receiveProcessOutput(QString,Utils::OutputFormat)));

src/plugins/valgrind/valgrindprocess.cpp

@@ -35,6 +35,7 @@
 #include <QFileInfo>
 
 #include <utils/qtcassert.h>
+#include <utils/qtcprocess.h>
 
 namespace Valgrind {
 
@@ -74,7 +75,7 @@ QString ValgrindProcess::workingDirectory() const
 bool ValgrindProcess::isRunning() const
 {
     if (isLocal())
-        return m_localProcess.state() != QProcess::NotRunning;
+        return m_localProcess.isRunning();
     else
         return m_remote.m_process && m_remote.m_process->isRunning();
 }
@@ -106,10 +107,15 @@ void ValgrindProcess::setEnvironment(const Utils::Environment &environment)
     ///TODO: remote anything that should/could be done here?
 }
 
+void ValgrindProcess::setLocalRunMode(ProjectExplorer::ApplicationLauncher::Mode localRunMode)
+{
+    m_localRunMode = localRunMode;
+}
+
 void ValgrindProcess::close()
 {
     if (isLocal()) {
-        m_localProcess.terminate();
+        m_localProcess.stop();
     } else {
         QTC_ASSERT(m_remote.m_connection->state() == QSsh::SshConnection::Connected, return);
         if (m_remote.m_process) {
@@ -131,22 +137,18 @@ void ValgrindProcess::close()
 void ValgrindProcess::run()
 {
     if (isLocal()) {
-        connect(&m_localProcess, SIGNAL(finished(int,QProcess::ExitStatus)),
+        connect(&m_localProcess, SIGNAL(processExited(int,QProcess::ExitStatus)),
                 this, SIGNAL(finished(int,QProcess::ExitStatus)));
-        connect(&m_localProcess, SIGNAL(started()),
+        connect(&m_localProcess, SIGNAL(processStarted()),
-                this, SIGNAL(started()));
+                this, SLOT(localProcessStarted()));
         connect(&m_localProcess, SIGNAL(error(QProcess::ProcessError)),
                 this, SIGNAL(error(QProcess::ProcessError)));
-        connect(&m_localProcess, SIGNAL(readyReadStandardError()),
+        connect(&m_localProcess, SIGNAL(appendMessage(QString,Utils::OutputFormat)),
-                this, SLOT(handleReadyReadStandardError()));
+                this, SIGNAL(processOutput(QString,Utils::OutputFormat)));
-        connect(&m_localProcess, SIGNAL(readyReadStandardOutput()),
+
-                this, SLOT(handleReadyReadStandardOutput()));
+        m_localProcess.start(m_localRunMode, m_valgrindExecutable,
+                             argumentString(Utils::HostOsInfo::hostOs()));
 
-        m_localProcess.setCommand(m_valgrindExecutable,
-                                  argumentString(Utils::HostOsInfo::hostOs()));
-        m_localProcess.start();
-        m_localProcess.waitForStarted();
-        m_pid = Utils::qPidToPid(m_localProcess.pid());
     } else {
         m_remote.m_valgrindExe = m_valgrindExecutable;
         m_remote.m_debuggee = m_debuggeeExecutable;
@@ -204,26 +206,18 @@ qint64 ValgrindProcess::pid() const
     return m_pid;
 }
 
-void ValgrindProcess::handleReadyReadStandardError()
+void ValgrindProcess::handleRemoteStderr()
 {
-    QByteArray b;
+    const QString b = QString::fromLocal8Bit(m_remote.m_process->readAllStandardError());
-    if (isLocal())
-        b = m_localProcess.readAllStandardError();
-    else
-        b = m_remote.m_process->readAllStandardError();
     if (!b.isEmpty())
-        emit processOutput(QString::fromLocal8Bit(b), Utils::StdErrFormat);
+        emit processOutput(b, Utils::StdErrFormat);
 }
 
-void ValgrindProcess::handleReadyReadStandardOutput()
+void ValgrindProcess::handleRemoteStdout()
 {
-    QByteArray b;
+    const QString b = QString::fromLocal8Bit(m_remote.m_process->readAllStandardOutput());
-    if (isLocal())
-        b = m_localProcess.readAllStandardOutput();
-    else
-        b = m_remote.m_process->readAllStandardOutput();
     if (!b.isEmpty())
-        emit processOutput(QString::fromLocal8Bit(b), Utils::StdOutFormat);
+        emit processOutput(b, Utils::StdOutFormat);
 }
 
 /// Remote
@@ -243,23 +237,28 @@ void ValgrindProcess::connected()
 
     m_remote.m_process = m_remote.m_connection->createRemoteProcess(cmd.toUtf8());
     connect(m_remote.m_process.data(), SIGNAL(readyReadStandardError()),
-            this, SLOT(handleReadyReadStandardError()));
+            this, SLOT(handleRemoteStderr()));
     connect(m_remote.m_process.data(), SIGNAL(readyReadStandardOutput()),
-            this, SLOT(handleReadyReadStandardOutput()));
+            this, SLOT(handleRemoteStdout()));
     connect(m_remote.m_process.data(), SIGNAL(closed(int)),
             this, SLOT(closed(int)));
     connect(m_remote.m_process.data(), SIGNAL(started()),
-            this, SLOT(processStarted()));
+            this, SLOT(remoteProcessStarted()));
     m_remote.m_process->start();
 }
 
-
 QSsh::SshConnection *ValgrindProcess::connection() const
 {
     return m_remote.m_connection;
 }
 
-void ValgrindProcess::processStarted()
+void ValgrindProcess::localProcessStarted()
+{
+    m_pid = m_localProcess.applicationPID();
+    emit started();
+}
+
+void ValgrindProcess::remoteProcessStarted()
 {
     QTC_ASSERT(m_remote.m_connection->state() == QSsh::SshConnection::Connected, return);
 
@@ -284,7 +283,7 @@ void ValgrindProcess::processStarted()
 
     m_remote.m_findPID = m_remote.m_connection->createRemoteProcess(cmd.toUtf8());
     connect(m_remote.m_findPID.data(), SIGNAL(readyReadStandardError()),
-            this, SLOT(handleReadyReadStandardError()));
+            this, SLOT(handleRemoteStderr()));
     connect(m_remote.m_findPID.data(), SIGNAL(readyReadStandardOutput()),
             this, SLOT(findPIDOutputReceived()));
     m_remote.m_findPID->start();

src/plugins/valgrind/valgrindprocess.h

@@ -31,7 +31,8 @@
 #ifndef VALGRINDPROCESS_H
 #define VALGRINDPROCESS_H
 
-#include <utils/qtcprocess.h>
+#include <projectexplorer/applicationlauncher.h>
+
 #include <ssh/sshremoteprocess.h>
 #include <ssh/sshconnection.h>
 #include <utils/osspecificaspects.h>
@@ -79,6 +80,7 @@ public:
     void setWorkingDirectory(const QString &path);
     QString workingDirectory() const;
     void setEnvironment(const Utils::Environment &environment);
+    void setLocalRunMode(ProjectExplorer::ApplicationLauncher::Mode localRunMode);
 
     qint64 pid() const;
     QSsh::SshConnection *connection() const;
@@ -92,19 +94,21 @@ signals:
     void localHostAddressRetrieved(const QHostAddress &localHostAddress);
 
 private slots:
-    void handleReadyReadStandardError();
+    void handleRemoteStderr();
-    void handleReadyReadStandardOutput();
+    void handleRemoteStdout();
     void handleError(QSsh::SshError);
 
     void closed(int);
     void connected();
-    void processStarted();
+    void localProcessStarted();
+    void remoteProcessStarted();
     void findPIDOutputReceived();
 
 private:
     QString argumentString(Utils::OsType osType) const;
 
-    Utils::QtcProcess m_localProcess;
+    ProjectExplorer::ApplicationLauncher m_localProcess;
+
     qint64 m_pid;
 
     Remote m_remote;
@@ -113,6 +117,7 @@ private:
     QString m_debuggeeExecutable;
     QString m_debuggeeArguments;
     bool m_isLocal;
+    ProjectExplorer::ApplicationLauncher::Mode m_localRunMode;
 };
 
 

src/plugins/valgrind/valgrindruncontrolfactory.cpp

@@ -94,6 +94,7 @@ RunControl *ValgrindRunControlFactory::create(RunConfiguration *runConfiguration
         sp.connParams.host = server.serverAddress().toString();
         sp.connParams.port = server.serverPort();
         sp.startMode = StartLocal;
+        sp.localRunMode = static_cast<ProjectExplorer::ApplicationLauncher::Mode>(rc1->runMode());
     } else if (RemoteLinux::AbstractRemoteLinuxRunConfiguration *rc2 =
                qobject_cast<RemoteLinux::AbstractRemoteLinuxRunConfiguration *>(runConfiguration)) {
         sp.startMode = StartRemote;

src/plugins/valgrind/valgrindrunner.cpp

@@ -50,7 +50,8 @@ public:
           process(0),
           channelMode(QProcess::SeparateChannels),
           finished(false),
-          startMode(Analyzer::StartLocal)
+          startMode(Analyzer::StartLocal),
+          localRunMode(ProjectExplorer::ApplicationLauncher::Gui)
     {
     }
 
@@ -67,6 +68,7 @@ public:
     QString debuggeeArguments;
     QString workingdir;
     Analyzer::StartMode startMode;
+    ProjectExplorer::ApplicationLauncher::Mode localRunMode;
     QSsh::SshConnectionParameters connParams;
 };
 
@@ -87,6 +89,7 @@ void ValgrindRunner::Private::run(ValgrindProcess *_process)
 
     process->setWorkingDirectory(workingdir);
     process->setProcessChannelMode(channelMode);
+    process->setLocalRunMode(localRunMode);
     // consider appending our options last so they override any interfering user-supplied options
     // -q as suggested by valgrind manual
 
@@ -179,6 +182,16 @@ Analyzer::StartMode ValgrindRunner::startMode() const
     return d->startMode;
 }
 
+void ValgrindRunner::setLocalRunMode(ProjectExplorer::ApplicationLauncher::Mode localRunMode)
+{
+    d->localRunMode = localRunMode;
+}
+
+ProjectExplorer::ApplicationLauncher::Mode ValgrindRunner::localRunMode() const
+{
+    return d->localRunMode;
+}
+
 void ValgrindRunner::setStartMode(Analyzer::StartMode startMode)
 {
     d->startMode = startMode;

src/plugins/valgrind/valgrindrunner.h

@@ -32,6 +32,7 @@
 #define VALGRIND_RUNNER_H
 
 #include <analyzerbase/analyzerconstants.h>
+#include <projectexplorer/applicationlauncher.h>
 
 #include <utils/outputformat.h>
 #include <ssh/sshconnection.h>
@@ -73,6 +74,9 @@ public:
     void setStartMode(Analyzer::StartMode startMode);
     Analyzer::StartMode startMode() const;
 
+    void setLocalRunMode(ProjectExplorer::ApplicationLauncher::Mode localRunMode);
+    ProjectExplorer::ApplicationLauncher::Mode localRunMode() const;
+
     void setConnectionParameters(const QSsh::SshConnectionParameters &connParams);
     const QSsh::SshConnectionParameters &connectionParameters() const;
 

tests/auto/valgrind/callgrind/callgrindparsertests.pro

@@ -1,4 +1,5 @@
 QTC_LIB_DEPENDS += utils ssh
+QTC_PLUGIN_DEPENDS += projectexplorer
 include(../../qttest.pri)
 include($$IDE_SOURCE_TREE/src/plugins/valgrind/valgrind_test.pri)
 TARGET = tst_callgrindparsertests

tests/auto/valgrind/memcheck/modeldemo.pro

@@ -1,4 +1,5 @@
 QTC_LIB_DEPENDS += utils ssh
+QTC_PLUGIN_DEPENDS += projectexplorer
 include(../../../../qtcreator.pri)
 include(../../qttestrpath.pri)
 include($$IDE_SOURCE_TREE/src/plugins/valgrind/valgrind_test.pri)

tests/auto/valgrind/memcheck/parsertests.pro

@@ -1,4 +1,5 @@
 QTC_LIB_DEPENDS += utils ssh
+QTC_PLUGIN_DEPENDS += projectexplorer
 include(../../qttest.pri)
 include($$IDE_SOURCE_TREE/src/plugins/valgrind/valgrind_test.pri)
 

tests/auto/valgrind/memcheck/testrunner.pro

@@ -1,4 +1,5 @@
 QTC_LIB_DEPENDS += utils ssh
+QTC_PLUGIN_DEPENDS += projectexplorer
 include(../../qttest.pri)
 include($$IDE_SOURCE_TREE/src/plugins/valgrind/valgrind_test.pri)
 

tests/auto/valgrind/valgrindautotest.qbs

@@ -4,6 +4,7 @@ import QtcAutotest
 QtcAutotest {
     Depends { name: "QtcSsh" }
     Depends { name: "Utils" }
+    Depends { name: "ProjectExplorer" }
     Depends { name: "Qt.widgets" } // TODO: Remove when qbs bug is fixed
     property path pluginDir: project.ide_source_tree + "/src/plugins/valgrind"