Do not write directly into logger

Pass information up to the client instead so that it can
decide how to handle it.

This allows e.g. creator to parse compile output for build
issues.

Change-Id: I76be4eaeb289b211b7d2527b555536b556655492
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>

1 files changed, 4 insertions, 2 deletions

diff --git a/src/lib/buildgraph/abstractcommandexecutor.h b/src/lib/buildgraph/abstractcommandexecutor.h
index 5e453ed77..1ba08759b 100644
--- a/src/lib/buildgraph/abstractcommandexecutor.h
+++ b/src/lib/buildgraph/abstractcommandexecutor.h
@@ -37,6 +37,8 @@ class QScriptEngine;
 QT_END_NAMESPACE
 
 namespace qbs {
+class Error;
+
 namespace Internal {
 class AbstractCommand;
 class Transformer;
@@ -56,7 +58,8 @@ public slots:
     void start(Transformer *transformer, const AbstractCommand *cmd);
 
 signals:
-    void error(QString errorString);
+    void reportCommandDescription(const QString &highlight, const QString &message);
+    void error(const qbs::Error &err);
     void finished();
 
 protected:
@@ -67,7 +70,6 @@ protected:
 
 private:
     virtual void doStart() = 0;
-    void printCommandInfo();
 
 private:
     const AbstractCommand *m_command;