diff --git a/plugins/execute/nativeappjob.h b/plugins/execute/nativeappjob.h --- a/plugins/execute/nativeappjob.h +++ b/plugins/execute/nativeappjob.h @@ -20,36 +20,27 @@ #ifndef KDEVPLATFORM_PLUGIN_NATIVEAPPJOB_H #define KDEVPLATFORM_PLUGIN_NATIVEAPPJOB_H +#include + #include -#include namespace KDevelop { class ILaunchConfiguration; -class ProcessLineMaker; -class OutputModel; } class KProcess; -class NativeAppJob : public KDevelop::OutputJob +class NativeAppJob : public KDevelop::OutputExecuteJob { -Q_OBJECT + Q_OBJECT + public: NativeAppJob( QObject* parent, KDevelop::ILaunchConfiguration* cfg ); + void start() override; - bool doKill() override; - KDevelop::OutputModel* model(); -private slots: - void processError(QProcess::ProcessError); - void processFinished(int,QProcess::ExitStatus); - void outputDone(); private: - void appendLine(const QString &l); - - KProcess* proc; - KDevelop::ProcessLineMaker* lineMaker; QString m_cfgname; }; diff --git a/plugins/execute/nativeappjob.cpp b/plugins/execute/nativeappjob.cpp --- a/plugins/execute/nativeappjob.cpp +++ b/plugins/execute/nativeappjob.cpp @@ -24,20 +24,15 @@ #include #include -#include #include #include -#include #include #include #include -#include -#include #include #include -#include #include #include @@ -47,7 +42,7 @@ using namespace KDevelop; NativeAppJob::NativeAppJob(QObject* parent, KDevelop::ILaunchConfiguration* cfg) - : KDevelop::OutputJob( parent ), proc(0) + : KDevelop::OutputExecuteJob( parent ) , m_cfgname(cfg->name()) { setCapabilities(Killable); @@ -75,6 +70,7 @@ "Using default environment group.", cfg->name() ); envgrp = l.defaultGroup(); } + setEnvironmentProfile(envgrp); QStringList arguments = iface->arguments( cfg, err ); if( !err.isEmpty() ) @@ -89,36 +85,21 @@ return; } - proc = new KProcess( this ); - - lineMaker = new KDevelop::ProcessLineMaker( proc, this ); - setStandardToolView(KDevelop::IOutputView::RunView); setBehaviours(KDevelop::IOutputView::AllowUserClose | KDevelop::IOutputView::AutoScroll); - OutputModel* m = new OutputModel; - m->setFilteringStrategy(OutputModel::NativeAppErrorFilter); - setModel(m); - setDelegate( new KDevelop::OutputDelegate ); - - connect( lineMaker, &ProcessLineMaker::receivedStdoutLines, model(), &OutputModel::appendLines ); - connect( proc, static_cast(&KProcess::error), this, &NativeAppJob::processError ); - connect( proc, static_cast(&KProcess::finished), this, &NativeAppJob::processFinished ); + setFilteringStrategy(OutputModel::NativeAppErrorFilter); + setProperties(DisplayStdout | DisplayStderr); // Now setup the process parameters - proc->setEnvironment( l.createEnvironment( envgrp, proc->systemEnvironment()) ); QUrl wc = iface->workingDirectory( cfg ); - if( !wc.isValid() || wc.isEmpty() ) - { + if( !wc.isValid() || wc.isEmpty() ) { wc = QUrl::fromLocalFile( QFileInfo( executable.toLocalFile() ).absolutePath() ); } - proc->setWorkingDirectory( wc.toLocalFile() ); - proc->setProperty( "executable", executable ); + setWorkingDirectory( wc ); qCDebug(PLUGIN_EXECUTE) << "setting app:" << executable << arguments; - proc->setOutputChannelMode(KProcess::MergedChannels); - if (iface->useTerminal(cfg)) { QStringList args = KShell::splitArgs(iface->terminal(cfg)); for (QStringList::iterator it = args.begin(); it != args.end(); ++it) { @@ -129,12 +110,13 @@ } } args.append( arguments ); - proc->setProgram( args ); + *this << args; } else { - proc->setProgram( executable.toLocalFile(), arguments ); + *this << executable.toLocalFile(); + *this << arguments; } - setObjectName(cfg->name()); + setJobName(cfg->name()); } NativeAppJob* findNativeJob(KJob* j) @@ -160,84 +142,5 @@ } } - qCDebug(PLUGIN_EXECUTE) << "launching?" << proc; - if( proc ) - { - startOutput(); - appendLine( i18n("Starting: %1", proc->program().join(" ") ) ); - proc->start(); - } else - { - qWarning() << "No process, something went wrong when creating the job"; - // No process means we've returned early on from the constructor, some bad error happened - emitResult(); - } -} - -bool NativeAppJob::doKill() -{ - if( proc ) { - proc->kill(); - appendLine( i18n( "*** Killed Application ***" ) ); - } - return true; -} - -void NativeAppJob::processFinished( int exitCode , QProcess::ExitStatus status ) -{ - if (!model()) { - outputDone(); - return; - } - - connect(model(), &OutputModel::allDone, this, &NativeAppJob::outputDone); - lineMaker->flushBuffers(); - - if (exitCode == 0 && status == QProcess::NormalExit) { - appendLine( i18n("*** Exited normally ***") ); - } else if (status == QProcess::NormalExit) { - appendLine( i18n("*** Exited with return code: %1 ***", QString::number(exitCode)) ); - setError(OutputJob::FailedShownError); - } else if (error() == KJob::KilledJobError) { - appendLine( i18n("*** Process aborted ***") ); - setError(KJob::KilledJobError); - } else { - appendLine( i18n("*** Crashed with return code: %1 ***", QString::number(exitCode)) ); - setError(OutputJob::FailedShownError); - } - - model()->ensureAllDone(); -} - -void NativeAppJob::outputDone() -{ - emitResult(); -} - -void NativeAppJob::processError( QProcess::ProcessError error ) -{ - if( error == QProcess::FailedToStart ) - { - setError( FailedShownError ); - QString errmsg = i18n("*** Could not start program '%1'. Make sure that the " - "path is specified correctly ***", proc->program().join(" ") ); - appendLine( errmsg ); - setErrorText( errmsg ); - emitResult(); - } - qCDebug(PLUGIN_EXECUTE) << "Process error"; + OutputExecuteJob::start(); } - -void NativeAppJob::appendLine(const QString& l) -{ - if (KDevelop::OutputModel* m = model()) { - m->appendLine(l); - } -} - -KDevelop::OutputModel* NativeAppJob::model() -{ - return dynamic_cast( OutputJob::model() ); -} - -