From c4451570e7de9bdd6bb57447ef22ea86f523ac00 Mon Sep 17 00:00:00 2001 From: BogDan Vatra Date: Tue, 24 Jun 2014 17:04:36 +0200 Subject: [PATCH] AndroidDeployQtStep::processFinished: Check for exitStatus and exitCode If the build failed there's no point in opening a shell. Change-Id: Icf27fa82c105f934a5d81ab0f317e0396cb12697 Reviewed-by: BogDan Vatra --- src/plugins/android/androiddeployqtstep.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/plugins/android/androiddeployqtstep.cpp b/src/plugins/android/androiddeployqtstep.cpp index 14bd592d37d..795b07ad67c 100644 --- a/src/plugins/android/androiddeployqtstep.cpp +++ b/src/plugins/android/androiddeployqtstep.cpp @@ -386,7 +386,7 @@ ProjectExplorer::BuildStepConfigWidget *AndroidDeployQtStep::createConfigWidget( void AndroidDeployQtStep::processFinished(int exitCode, QProcess::ExitStatus status) { AbstractProcessStep::processFinished(exitCode, status); - if (m_openPackageLocationForRun) + if (m_openPackageLocationForRun && status == QProcess::NormalExit && exitCode == 0) QMetaObject::invokeMethod(this, "showInGraphicalShell", Qt::QueuedConnection); }