diff options
author | Tim Jenssen <tim.jenssen@qt.io> | 2019-07-04 14:47:52 +0200 |
---|---|---|
committer | Tim Jenssen <tim.jenssen@qt.io> | 2019-07-04 15:17:55 +0200 |
commit | 1df6443d4a6e58338dfaed7d787debfaeba86756 (patch) | |
tree | 29ceb6ea3313311287eb5b5216a8a49c1ec0ecfd /src/libs/utils/synchronousprocess.cpp | |
parent | 2b7fab166878b759fc8fa29256754a41c3fc6259 (diff) | |
parent | 6a58666f4403cdde1809784c1cc98668a4b02cfc (diff) |
Merge remote-tracking branch 'origin/4.10'
Conflicts:
qbs/modules/qtc/qtc.qbs
qtcreator_ide_branding.pri
src/libs/utils/synchronousprocess.cpp
src/plugins/baremetal/iarewtoolchain.cpp
src/plugins/cmakeprojectmanager/cmakeproject.cpp
tests/unit/unittest/CMakeLists.txt
Change-Id: I124ad492df403286751e175d27fe36487ddf6d07
Diffstat (limited to 'src/libs/utils/synchronousprocess.cpp')
-rw-r--r-- | src/libs/utils/synchronousprocess.cpp | 25 |
1 files changed, 17 insertions, 8 deletions
diff --git a/src/libs/utils/synchronousprocess.cpp b/src/libs/utils/synchronousprocess.cpp index 82004b87293..f197494b4c1 100644 --- a/src/libs/utils/synchronousprocess.cpp +++ b/src/libs/utils/synchronousprocess.cpp @@ -24,16 +24,18 @@ ****************************************************************************/ #include "synchronousprocess.h" -#include "qtcassert.h" +#include "executeondestruction.h" #include "hostosinfo.h" -#include "fileutils.h" +#include "qtcassert.h" +#include "qtcprocess.h" #include <QDebug> -#include <QTimer> -#include <QTextCodec> #include <QDir> +#include <QLoggingCategory> #include <QMessageBox> +#include <QTextCodec> #include <QThread> +#include <QTimer> #include <QApplication> @@ -82,6 +84,8 @@ enum { defaultMaxHangTimerCount = 10 }; namespace Utils { +Q_LOGGING_CATEGORY(processLog, "qtc.utils.synchronousprocess", QtWarningMsg); + // A special QProcess derivative allowing for terminal control. class TerminalControllingProcess : public QProcess { public: @@ -445,8 +449,10 @@ static bool isGuiThread() SynchronousProcessResponse SynchronousProcess::run(const CommandLine &cmd, const QByteArray &writeData) { - if (debug) - qDebug() << '>' << Q_FUNC_INFO << cmd.toUserOutput(); + qCDebug(processLog).noquote() << "Starting:" << cmd.toUserOutput(); + ExecuteOnDestruction logResult([this] { + qCDebug(processLog) << d->m_result; + }); d->clearForRun(); @@ -492,13 +498,16 @@ SynchronousProcessResponse SynchronousProcess::run(const CommandLine &cmd, QApplication::restoreOverrideCursor(); } - if (debug) - qDebug() << '<' << Q_FUNC_INFO << cmd.executable().toString() << d->m_result; return d->m_result; } SynchronousProcessResponse SynchronousProcess::runBlocking(const CommandLine &cmd) { + qCDebug(processLog).noquote() << "Starting blocking:" << cmd.toUserOutput(); + ExecuteOnDestruction logResult([this] { + qCDebug(processLog) << d->m_result; + }); + d->clearForRun(); // On Windows, start failure is triggered immediately if the |