From e63fb8419b0968f5a6ebb6c468000b2cf4811d5e Mon Sep 17 00:00:00 2001
From: Thomas Perl <thomas.perl@jolla.com>
Date: Tue, 15 Apr 2014 10:53:54 +0200
Subject: [PATCH] [ssudeviceinfo] Remove obsolete QtSystems dependency

Our QtSystems depends on libssu now for the device model,
so we should not make QtSystems depend on libssu to avoid
infinite recursion.
---
 board-mappings.ini                            |  9 +-------
 libssu/ssudeviceinfo.cpp                      | 23 -------------------
 .../ut_deviceinfo/testdata/board-mappings.ini |  2 --
 .../ut_rndssucli/testdata/board-mappings.ini  |  2 --
 .../testdata/board-mappings.ini               |  2 --
 .../testdata/board-mappings.ini               |  2 --
 6 files changed, 1 insertion(+), 39 deletions(-)

diff --git a/board-mappings.ini b/board-mappings.ini
index 2555ad7..9a20d24 100644
--- a/board-mappings.ini
+++ b/board-mappings.ini
@@ -17,18 +17,13 @@
 # - boardname.contains -- searches for substring in boardname
 #   NOTE: only use boardname if none of the other options match
 #         use the boardname command to set/check the current value
-# - systeminfo.equals -- compares the model returned by QSystemInfo
-#                        with the value provided
-#   NOTE: systeminfo matching is currently disabled, as this would
-#         pull in the whole X11 stack, and on Mer/Nemo does cpuinfo
-#         matching only anyway, which ssu can do better already.
 # - cpuinfo.contains -- searches /proc/cpuinfo for a string
 # - uname-release.contains -- searches the kernels release string for
 #         a string (uname -r)
 # - arch.equals -- compares with zyppers arch (like i586)
 #
 # Resolve order is:
-#  file.exists -> systeminfo.equals -> cpuinfo.contains
+#  file.exists -> cpuinfo.contains
 #   -> uname-release.contains -> arch.equals
 #
 # The found model (after resolving variants) will be used as category. The
@@ -85,8 +80,6 @@
 [file.exists]
 SDK=/mer-sdk-chroot
 
-[systeminfo.equals]
-
 [cpuinfo.contains]
 N900=Nokia RX-51 board
 N950=Nokia RM-680 board
diff --git a/libssu/ssudeviceinfo.cpp b/libssu/ssudeviceinfo.cpp
index 4cf8544..d71130b 100644
--- a/libssu/ssudeviceinfo.cpp
+++ b/libssu/ssudeviceinfo.cpp
@@ -201,29 +201,6 @@ QString SsuDeviceInfo::deviceModel(){
   }
   if (!cachedModel.isEmpty()) return cachedModel;
 
-  // check if the QSystemInfo model is useful
-  //QSystemDeviceInfo devInfo;
-  // TODO Current Mer SystemDeviceInfo only returns cpuinfo stuff,
-  //      which is what we can do with cpuinfo matching in a more
-  //      flexible way, so there's not really any need to pull in the
-  //      whole X11 stack just for this. Can be enabled once systeminfo
-  //      is less insane
-  /*
-  QSystemDeviceInfoLinuxCommonPrivate devInfo;
-  QString model = devInfo.model();
-  boardMappings->beginGroup("systeminfo.equals");
-  keys = boardMappings->allKeys();
-  foreach (const QString &key, keys){
-    QString value = boardMappings->value(key).toString();
-    if (model == value){
-      cachedModel = key;
-      break;
-    }
-  }
-  boardMappings->endGroup();
-  if (!cachedModel.isEmpty()) return cachedModel;
-  */
-
   // check if the device can be identified by a string in /proc/cpuinfo
   procCpuinfo.setFileName(Sandbox::map("/proc/cpuinfo"));
   procCpuinfo.open(QIODevice::ReadOnly | QIODevice::Text);
diff --git a/tests/ut_deviceinfo/testdata/board-mappings.ini b/tests/ut_deviceinfo/testdata/board-mappings.ini
index 4648fce..e651055 100644
--- a/tests/ut_deviceinfo/testdata/board-mappings.ini
+++ b/tests/ut_deviceinfo/testdata/board-mappings.ini
@@ -1,8 +1,6 @@
 [file.exists]
 SDK=/mer-sdk-chroot
 
-[systeminfo.equals]
-
 [cpuinfo.contains]
 N900=Nokia RX-51 board
 N950=Nokia RM-680 board
diff --git a/tests/ut_rndssucli/testdata/board-mappings.ini b/tests/ut_rndssucli/testdata/board-mappings.ini
index 49c0e66..6dad6b5 100644
--- a/tests/ut_rndssucli/testdata/board-mappings.ini
+++ b/tests/ut_rndssucli/testdata/board-mappings.ini
@@ -1,8 +1,6 @@
 [file.exists]
 SDK=/mer-sdk-chroot
 
-[systeminfo.equals]
-
 [cpuinfo.contains]
 N900=Nokia RX-51 board
 N950=Nokia RM-680 board
diff --git a/tests/ut_ssuurlresolver/testdata/board-mappings.ini b/tests/ut_ssuurlresolver/testdata/board-mappings.ini
index 49c0e66..6dad6b5 100644
--- a/tests/ut_ssuurlresolver/testdata/board-mappings.ini
+++ b/tests/ut_ssuurlresolver/testdata/board-mappings.ini
@@ -1,8 +1,6 @@
 [file.exists]
 SDK=/mer-sdk-chroot
 
-[systeminfo.equals]
-
 [cpuinfo.contains]
 N900=Nokia RX-51 board
 N950=Nokia RM-680 board
diff --git a/tests/ut_urlresolver/testdata/board-mappings.ini b/tests/ut_urlresolver/testdata/board-mappings.ini
index 49c0e66..6dad6b5 100644
--- a/tests/ut_urlresolver/testdata/board-mappings.ini
+++ b/tests/ut_urlresolver/testdata/board-mappings.ini
@@ -1,8 +1,6 @@
 [file.exists]
 SDK=/mer-sdk-chroot
 
-[systeminfo.equals]
-
 [cpuinfo.contains]
 N900=Nokia RX-51 board
 N950=Nokia RM-680 board