Remove deprecated udisks plugin

udisks and dbus-glib have been deprecated for at least 5 years.
Linux Mint 17.3 and therefore Ubuntu 14.04 LTS ship with udisks2 as default.
I've also made a new plugin for udisks2.
Let's just get rid of this, shall we?

5 files changed, 0 insertions, 428 deletions

diff --git a/configure.ac b/configure.ac
index 9261f77..017c3de 100644
--- a/configure.ac
+++ b/configure.ac
@@ -52,8 +52,6 @@ GTK_REQUIRED=3.14.0
 LIBPANEL_REQUIRED=1.17.0
 LIBNOTIFY_REQUIRED=0.7.0
 LIBCAIRO_REQUIRED=1.0.4
-LIBDBUSGLIB_REQUIRED=0.80
-LIBATASMART_REQUIRED=0.16
 
 PKG_CHECK_MODULES(GLIB, glib-2.0 >= $GLIB_REQUIRED gio-2.0)
 AC_SUBST(GLIB_CFLAGS)
@@ -71,46 +69,9 @@ PKG_CHECK_MODULES(CAIRO, cairo >= $LIBCAIRO_REQUIRED)
 AC_SUBST(CAIRO_CFLAGS)
 AC_SUBST(CAIRO_LIBS)
 
-# enable support for udisks (via dbus) by default if its available
-AC_ARG_ENABLE(udisks,
-	      AS_HELP_STRING([--disable-udisks],
-			     [Disable support for udisks sensor readings via DBUS.]),
-			     enable_udisks=$enableval, enable_udisks="yes")
-
-if test "x$enable_udisks" = "xno" ; then
-	echo "Disabling udisks support"
-else
-	PKG_CHECK_MODULES(DBUSGLIB, dbus-glib-1 >= $LIBDBUSGLIB_REQUIRED,
-		          HAVE_DBUSGLIB="yes", HAVE_DBUSGLIB="no")
-	AC_SUBST(DBUSGLIB_CFLAGS)
-	AC_SUBST(DBUSGLIB_LIBS)
-
-	PKG_CHECK_MODULES(LIBATASMART, libatasmart >= $LIBATASMART_REQUIRED,
-			  HAVE_LIBATASMART="yes", HAVE_LIBATASMART="no")
-	AC_SUBST(LIBATASMART_CFLAGS)
-	AC_SUBST(LIBATASMART_LIBS)
-fi
-
-AM_CONDITIONAL(UDISKS, test -n "$DBUSGLIB_LIBS" -a -n "$LIBATASMART_LIBS")
-
-# enable support for udisks2 (via dbus) by default if its available
-AC_ARG_ENABLE(udisks2,
-	      AS_HELP_STRING([--disable-udisks2],
-			     [Disable support for udisks2 sensor readings via DBUS.]),
-			     enable_udisks2=$enableval, enable_udisks2="yes")
 
 if test "x$enable_udisks2" = "xno" ; then
 	echo "Disabling udisks2 support"
-#else
-#	PKG_CHECK_MODULES(DBUSGLIB, dbus-glib-1 >= $LIBDBUSGLIB_REQUIRED,
-#		          HAVE_DBUSGLIB="yes", HAVE_DBUSGLIB="no")
-#	AC_SUBST(DBUSGLIB_CFLAGS)
-#	AC_SUBST(DBUSGLIB_LIBS)
-
-#	PKG_CHECK_MODULES(LIBATASMART, libatasmart >= $LIBATASMART_REQUIRED,
-#			  HAVE_LIBATASMART="yes", HAVE_LIBATASMART="no")
-#	AC_SUBST(LIBATASMART_CFLAGS)
-#	AC_SUBST(LIBATASMART_LIBS)
 fi
 
 # use same test as above, bc AM_CONDITIONAL may not be in if/else
@@ -290,7 +251,6 @@ AC_CONFIG_FILES([
 	lib/Makefile
 	plugins/Makefile
 	plugins/acpi/Makefile
-	plugins/udisks/Makefile
 	plugins/udisks2/Makefile
 	plugins/hddtemp/Makefile
 	plugins/i2c-proc/Makefile
diff --git a/plugins/Makefile.am b/plugins/Makefile.am
index f0e46a1..619faf3 100644
--- a/plugins/Makefile.am
+++ b/plugins/Makefile.am
@@ -1,6 +1,5 @@
 DIST_SUBDIRS = \
 	acpi \
-	udisks \
 	udisks2 \
 	hddtemp \
         i2c-proc \
@@ -27,10 +26,6 @@ SUBDIRS = \
 	smu-sys \
 	sonypi
 
-if UDISKS
-SUBDIRS	     += udisks
-endif
-
 if UDISKS2
 SUBDIRS	     += udisks2
 endif
diff --git a/plugins/udisks/Makefile.am b/plugins/udisks/Makefile.am
deleted file mode 100644
index 942e22d..0000000
--- a/plugins/udisks/Makefile.am
+++ /dev/null
@@ -1,25 +0,0 @@
-# UDisks plugin
-plugindir = $(libdir)/mate-sensors-applet/plugins
-
-AM_CPPFLAGS = -DMATELOCALEDIR=\""$(datadir)/locale/"\" \
-	-DG_LOG_DOMAIN=\""Sensors Applet"\" \
-	-DPIXMAPS_DIR=\""$(datadir)/pixmaps/$(PACKAGE)/"\" \
-	-DDATADIR=\""$(datadir)"\" \
-	-DLIBDIR=\""$(libdir)"\" \
-        -DSYSCONFDIR=\""$(sysconfdir)"\" \
-        -DPREFIX=\""$(prefix)"\" \
-	-I$(top_srcdir) \
-	$(DBUSGLIB_CFLAGS)
-	$(LIBATASMART_CFLAGS)
-
-
-
-plugin_LTLIBRARIES = libudisks.la
-
-libudisks_la_SOURCES = \
-	udisks-plugin.h	\
-	udisks-plugin.c
-
-libudisks_la_LDFLAGS = $(PLUGIN_LIBTOOL_FLAGS) $(DBUSGLIB_LIBS)	$(LIBATASMART_LIBS)
-
-libudisks_la_LIBADD = $(top_builddir)/lib/libmate-sensors-applet-plugin.la
diff --git a/plugins/udisks/udisks-plugin.c b/plugins/udisks/udisks-plugin.c
deleted file mode 100644
index 4f2bc7a..0000000
--- a/plugins/udisks/udisks-plugin.c
+++ /dev/null
@@ -1,334 +0,0 @@
-/*
- * Copyright (C) 2009 Pramod Dematagoda <[email protected]>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
- */
-
-#ifdef HAVE_CONFIG_H
-#include "config.h"
-#endif /* HAVE_CONFIG_H */
-
-#include <stdio.h>
-#include <atasmart.h>
-#include <glib.h>
-#include <dbus/dbus-glib.h>
-#include "udisks-plugin.h"
-
-#define UDISKS_BUS_NAME              "org.freedesktop.UDisks"
-#define UDISKS_DEVICE_INTERFACE_NAME "org.freedesktop.UDisks.Device"
-#define UDISKS_INTERFACE_NAME        "org.freedesktop.UDisks"
-#define UDISKS_PROPERTIES_INTERFACE  "org.freedesktop.DBus.Properties"
-#define UDISKS_OBJECT_PATH           "/org/freedesktop/UDisks"
-
-
-/*
- * Info about a single sensor
- */
-typedef struct _DevInfo{
-    gchar *path;
-    gchar *id;
-    gdouble temp;
-    DBusGProxy *sensor_proxy;
-    GError *error;
-} DevInfo;
-
-const gchar *plugin_name = "udisks";
-
-GHashTable *devices = NULL;
-
-/* This is a global variable for convenience */
-DBusGConnection *connection;
-
-static void update_device(DevInfo *info)
-{
-    DBusGProxy *sensor_proxy;
-    GValue smart_time = G_VALUE_INIT;
-    SkDisk *sk_disk;
-    GValue smart_blob_val = G_VALUE_INIT;
-    GArray *smart_blob;
-    guint64 temperature;
-
-    g_return_if_fail(info != NULL);
-
-    g_clear_error(&info->error);
-
-    sensor_proxy = dbus_g_proxy_new_for_name(connection,
-                                         UDISKS_BUS_NAME,
-                                         info->path,
-                                         UDISKS_PROPERTIES_INTERFACE);
-
-    if (!dbus_g_proxy_call(sensor_proxy, "Get", NULL,
-                           G_TYPE_STRING, UDISKS_BUS_NAME,
-                           G_TYPE_STRING, "DriveAtaSmartTimeCollected", G_TYPE_INVALID,
-                           G_TYPE_VALUE, &smart_time,
-                           G_TYPE_INVALID) ||
-        !g_value_get_uint64(&smart_time)) {
-
-        g_object_unref(sensor_proxy);
-        return;
-    }
-
-    if (!dbus_g_proxy_call(sensor_proxy, "Get", &info->error,
-                          G_TYPE_STRING, UDISKS_BUS_NAME,
-                          G_TYPE_STRING, "DriveAtaSmartBlob", G_TYPE_INVALID,
-                          G_TYPE_VALUE, &smart_blob_val,
-                          G_TYPE_INVALID)) {
-
-        g_debug("Error getting DriveAtaSmartBlob %s", info->error ? info->error->message : "NULL");
-        g_object_unref(sensor_proxy);
-        return;
-    }
-    smart_blob = g_value_get_boxed(&smart_blob_val);
-
-    sk_disk_open(NULL, &sk_disk);
-    sk_disk_set_blob(sk_disk, smart_blob->data, smart_blob->len);
-    if (sk_disk_smart_get_temperature(sk_disk, &temperature) < 0) {
-        g_debug("Error getting temperature from AtaSmartBlob");
-        g_free(sk_disk);
-        g_array_free(smart_blob, TRUE);
-        g_object_unref(sensor_proxy);
-        return;
-    }
-
-    /* Temperature is in mK, so convert it to K first */
-    temperature /= 1000;
-    info->temp = (gdouble)temperature - 273.15;
-
-    g_free(sk_disk);
-    g_array_free(smart_blob, TRUE);
-    g_object_unref(sensor_proxy);
-}
-
-/* This is the handler for the Changed() signal emitted by UDisks. */
-static void udisks_changed_signal_cb(DBusGProxy *sensor_proxy) {
-    const gchar *path;
-    DevInfo *info;
-
-    path = dbus_g_proxy_get_path(sensor_proxy);
-    g_debug("%s changed()", path);
-    info = g_hash_table_lookup(devices, path);
-
-    update_device(info);
-}
-
-static void udisks_plugin_get_sensors(GList **sensors) {
-    DBusGProxy *proxy, *sensor_proxy;
-    GError *error = NULL;
-    GPtrArray *paths;
-    guint i;
-    DevInfo *info;
-
-    /* This connection will be used for everything, including the obtaining of sensor data */
-    connection = dbus_g_bus_get(DBUS_BUS_SYSTEM, &error);
-    if (connection == NULL)
-    {
-        g_debug("Failed to open connection to DBUS: %s", error->message);
-        g_error_free(error);
-        return;
-    }
-
-    /* This is the proxy which is only used once during the enumeration of the device object paths */
-    proxy = dbus_g_proxy_new_for_name(connection,
-                                      UDISKS_BUS_NAME,
-                                      UDISKS_OBJECT_PATH,
-                                      UDISKS_INTERFACE_NAME);
-
-    /* The object paths of the disks are enumerated and placed in an array of object paths */
-    if (!dbus_g_proxy_call(proxy, "EnumerateDevices", &error,
-                            G_TYPE_INVALID,
-                            dbus_g_type_get_collection("GPtrArray",
-                            DBUS_TYPE_G_OBJECT_PATH),
-                            &paths,
-                            G_TYPE_INVALID)) {
-
-        g_debug("Failed to enumerate disk devices: %s", error->message);
-        g_error_free(error);
-        g_object_unref(proxy);
-        dbus_g_connection_unref(connection);
-        connection = NULL;
-        return;
-    }
-
-    for (i = 0; i < paths->len; i++) {
-        /* This proxy is used to get the required data in order to build up the list of sensors */
-        GValue smart_available = G_VALUE_INIT;
-        gchar *path = (gchar *)g_ptr_array_index(paths, i);
-
-        sensor_proxy = dbus_g_proxy_new_for_name(connection,
-                                                 UDISKS_BUS_NAME,
-                                                 path,
-                                                 UDISKS_PROPERTIES_INTERFACE);
-
-        if (dbus_g_proxy_call(sensor_proxy, "Get", &error,
-                              G_TYPE_STRING,
-                              UDISKS_BUS_NAME,
-                              G_TYPE_STRING,
-                              "DriveAtaSmartIsAvailable",
-                              G_TYPE_INVALID,
-                              G_TYPE_VALUE, &smart_available, G_TYPE_INVALID)) {
-
-            if (!g_value_get_boolean(&smart_available)) {
-                g_debug("Drive at path '%s' does not support Smart monitoring... ignoring", path);
-                g_object_unref(sensor_proxy);
-                g_free (path);
-                continue;
-            }
-
-            GValue model_v = G_VALUE_INIT;
-            dbus_g_proxy_call(sensor_proxy, "Get", NULL,
-                              G_TYPE_STRING, UDISKS_BUS_NAME,
-                              G_TYPE_STRING, "DriveModel",
-                              G_TYPE_INVALID,
-                              G_TYPE_VALUE, &model_v,
-                              G_TYPE_INVALID);
-
-            GValue dev_v = G_VALUE_INIT;
-            dbus_g_proxy_call(sensor_proxy, "Get", NULL,
-                              G_TYPE_STRING, UDISKS_BUS_NAME,
-                              G_TYPE_STRING, "DeviceFile",
-                              G_TYPE_INVALID,
-                              G_TYPE_VALUE, &dev_v,
-                              G_TYPE_INVALID);
-
-            GValue ids_v = G_VALUE_INIT;
-            dbus_g_proxy_call(sensor_proxy, "Get", NULL,
-                              G_TYPE_STRING, UDISKS_BUS_NAME,
-                              G_TYPE_STRING, "DeviceFileById",
-                              G_TYPE_INVALID,
-                              G_TYPE_VALUE, &ids_v,
-                              G_TYPE_INVALID);
-
-            g_object_unref(sensor_proxy);
-
-            sensor_proxy = dbus_g_proxy_new_for_name(connection,
-                                                     UDISKS_BUS_NAME,
-                                                     path,
-                                                     UDISKS_DEVICE_INTERFACE_NAME);
-
-            /* Use the Changed() signal emitted from UDisks to
-             * get the temperature immediately if it changes rather
-             * than waiting to poll
-             */
-            dbus_g_proxy_add_signal(sensor_proxy, "Changed", G_TYPE_INVALID);
-
-            dbus_g_proxy_connect_signal(sensor_proxy, "Changed",
-                                        G_CALLBACK(udisks_changed_signal_cb),
-                                        path, NULL);
-
-            const gchar *model = g_value_get_string(&model_v);
-            const gchar *dev = g_value_get_string(&dev_v);
-            const GStrv ids = g_value_get_boxed(&ids_v);
-
-            const gchar *id = ids != NULL && ids[0] != NULL ? ids[0] : dev;
-
-            info = g_malloc0(sizeof(DevInfo));
-            if (devices == NULL)
-            {
-                devices = g_hash_table_new(g_str_hash, g_str_equal);
-            }
-            info->id = g_strdup(id);
-            info->path = g_strdup(path);
-            info->sensor_proxy = sensor_proxy;
-            /* Set the device status changed as TRUE because we need
-             * to get the initial temperature reading
-             */
-            info->temp = 0.0;
-            g_hash_table_insert(devices, info->id, info);
-
-            /* Write the sensor data */
-            sensors_applet_plugin_add_sensor(sensors,
-                                             id,
-                                             "Disk Temperature",
-                                             model,
-                                             TEMP_SENSOR,
-                                             FALSE,
-                                             HDD_ICON,
-                                             DEFAULT_GRAPH_COLOR);
-
-            g_value_unset(&ids_v);
-            g_value_unset(&model_v);
-            g_value_unset(&dev_v);
-            g_debug("Added %s %s", path, id);
-        } else {
-            g_debug ("Cannot obtain data for device: %s\n"
-                 "Error: %s\n",
-                 path,
-                 error->message);
-            g_error_free (error);
-            error = NULL;
-            g_object_unref(sensor_proxy);
-        }
-        g_free(path);
-    } /* end of for */
-
-    g_ptr_array_free(paths, TRUE);
-    g_object_unref(proxy);
-    if (devices == NULL)
-    {
-        dbus_g_connection_unref(connection);
-        connection = NULL;
-    }
-}
-
-static gdouble udisks_plugin_get_sensor_value(const gchar *path,
-                                              const gchar *id,
-                                              SensorType type,
-                                              GError **error) {
-    DevInfo *info;
-
-    info = (DevInfo *)g_hash_table_lookup(devices, path);
-    if (info == NULL)
-    {
-        g_set_error(error, SENSORS_APPLET_PLUGIN_ERROR, 0, "Error finding disk with path %s", path);
-        return 0.0;
-    }
-
-    if (info->error)
-    {
-        *error = info->error;
-        info->error = NULL;
-        return 0.0;
-    }
-
-    /* update value since Changed() signal doesn't fire manually enough so poll instead */
-    update_device(info);
-    return info->temp;
-}
-
-static GList *udisks_plugin_init(void) {
-    GList *sensors = NULL;
-
-    udisks_plugin_get_sensors(&sensors);
-
-    return sensors;
-}
-
-const gchar *sensors_applet_plugin_name(void)
-{
-    return plugin_name;
-}
-
-GList *sensors_applet_plugin_init(void)
-{
-    return udisks_plugin_init();
-}
-
-gdouble sensors_applet_plugin_get_sensor_value(const gchar *path,
-                                               const gchar *id,
-                                               SensorType type,
-                                               GError **error) {
-
-    return udisks_plugin_get_sensor_value(path, id, type, error);
-}
diff --git a/plugins/udisks/udisks-plugin.h b/plugins/udisks/udisks-plugin.h
deleted file mode 100644
index bbc6a3e..0000000
--- a/plugins/udisks/udisks-plugin.h
+++ /dev/null
@@ -1,24 +0,0 @@
-/*
- * Copyright (C) 2009 Pramod Dematagoda <[email protected]>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
- */
-
-#ifndef UDISKS_PLUGIN_H
-#define UDISKS_PLUGIN_H
-
-#include <sensors-applet/sensors-applet-plugin.h>
-
-#endif /* UDISKS_PLUGIN_H */