Unify handling of dumperLibrary/dumperLibraryLocations

across its uses. We have the kits now, so we can use the information
from there and no longer need to guess at the dumpers to use.

Change-Id: I058304198e9c7fdbad45a84658e1bcea8ed9834d
Reviewed-by: Daniel Teske <daniel.teske@digia.com>

3 files changed, 24 insertions, 7 deletions

diff --git a/src/plugins/qtsupport/customexecutablerunconfiguration.cpp b/src/plugins/qtsupport/customexecutablerunconfiguration.cpp
index 807488aa6e..b13863ca6d 100644
--- a/src/plugins/qtsupport/customexecutablerunconfiguration.cpp
+++ b/src/plugins/qtsupport/customexecutablerunconfiguration.cpp
@@ -29,7 +29,7 @@
 
 #include "customexecutablerunconfiguration.h"
 #include "customexecutableconfigurationwidget.h"
-#include "debugginghelper.h"
+#include "qtkitinformation.h"
 
 #include <projectexplorer/buildconfiguration.h>
 #include <projectexplorer/project.h>
@@ -365,16 +365,12 @@ QWidget *CustomExecutableRunConfiguration::createConfigurationWidget()
 
 QString CustomExecutableRunConfiguration::dumperLibrary() const
 {
-    Utils::FileName qmakePath = DebuggingHelperLibrary::findSystemQt(environment());
-    QString qtInstallData = DebuggingHelperLibrary::qtInstallDataDir(qmakePath);
-    return DebuggingHelperLibrary::debuggingHelperLibraryByInstallData(qtInstallData);
+    return QtKitInformation::dumperLibrary(target()->kit());
 }
 
 QStringList CustomExecutableRunConfiguration::dumperLibraryLocations() const
 {
-    Utils::FileName qmakePath = DebuggingHelperLibrary::findSystemQt(environment());
-    QString qtInstallData = DebuggingHelperLibrary::qtInstallDataDir(qmakePath);
-    return DebuggingHelperLibrary::debuggingHelperLibraryDirectories(qtInstallData);
+    return QtKitInformation::dumperLibraryLocations(target()->kit());
 }
 
 ProjectExplorer::Abi CustomExecutableRunConfiguration::abi() const
diff --git a/src/plugins/qtsupport/qtkitinformation.cpp b/src/plugins/qtsupport/qtkitinformation.cpp
index d3ffd56352..766c9ea5d3 100644
--- a/src/plugins/qtsupport/qtkitinformation.cpp
+++ b/src/plugins/qtsupport/qtkitinformation.cpp
@@ -180,6 +180,22 @@ void QtKitInformation::setQtVersion(ProjectExplorer::Kit *k, const BaseQtVersion
         setQtVersionId(k, v->uniqueId());
 }
 
+QString QtKitInformation::dumperLibrary(const ProjectExplorer::Kit *k)
+{
+    BaseQtVersion *version = QtKitInformation::qtVersion(k);
+    if (version)
+        return version->gdbDebuggingHelperLibrary();
+    return QString();
+}
+
+QStringList QtKitInformation::dumperLibraryLocations(const ProjectExplorer::Kit *k)
+{
+    BaseQtVersion *version = QtKitInformation::qtVersion(k);
+    if (version)
+        return version->debuggingHelperLibraryLocations();
+    return QStringList();
+}
+
 void QtKitInformation::qtVersionsChanged(const QList<int> &addedIds,
                                          const QList<int> &removedIds,
                                          const QList<int> &changedIds)
diff --git a/src/plugins/qtsupport/qtkitinformation.h b/src/plugins/qtsupport/qtkitinformation.h
index b4ae29d6f4..8a4391a107 100644
--- a/src/plugins/qtsupport/qtkitinformation.h
+++ b/src/plugins/qtsupport/qtkitinformation.h
@@ -69,6 +69,11 @@ public:
     static BaseQtVersion *qtVersion(const ProjectExplorer::Kit *k);
     static void setQtVersion(ProjectExplorer::Kit *k, const BaseQtVersion *v);
 
+    // Information derived from the Qt version:
+    // FIXME: This should be part of an RunConfigurationAspect...
+    static QString dumperLibrary(const ProjectExplorer::Kit *k);
+    static QStringList dumperLibraryLocations(const ProjectExplorer::Kit *k);
+
 private slots:
     void qtVersionsChanged(const QList<int> &addedIds,
                            const QList<int> &removedIds,