From 0ea6c2853010c71fb66ea6ec0cf9d9540db50c09 Mon Sep 17 00:00:00 2001 From: Ted Gould Date: Thu, 28 May 2009 10:19:48 -0500 Subject: Failing on warnings --- src/Makefile.am | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index c543e48..74db2df 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -4,10 +4,10 @@ bin_PROGRAMS = indicator-messages-service messaginglibdir = $(INDICATORDIR) messaginglib_LTLIBRARIES = libmessaging.la libmessaging_la_SOURCES = indicator-messages.c im-menu-item.c im-menu-item.h app-menu-item.c app-menu-item.h -libmessaging_la_CFLAGS = $(APPLET_CFLAGS) -Wall -Wl,-Bsymbolic-functions -Wl,-z,defs -Wl,--as-needed +libmessaging_la_CFLAGS = $(APPLET_CFLAGS) -Wall -Wl,-Bsymbolic-functions -Wl,-z,defs -Wl,--as-needed -Werror libmessaging_la_LIBADD = $(APPLET_LIBS) libmessaging_la_LDFLAGS = -module -avoid-version indicator_messages_service_SOURCES = indicator-service.c im-menu-item.c im-menu-item.h app-menu-item.c app-menu-item.h -indicator_messages_service_CFLAGS = $(APPLET_CFLAGS) -Wall -Wl,-Bsymbolic-functions -Wl,-z,defs -Wl,--as-needed +indicator_messages_service_CFLAGS = $(APPLET_CFLAGS) -Wall -Wl,-Bsymbolic-functions -Wl,-z,defs -Wl,--as-needed -Werror indicator_messages_service_LDADD = $(APPLET_LIBS) -- cgit v1.2.3 From 5dd6b6286dd3299403d742f9cfd867f6b0aefba5 Mon Sep 17 00:00:00 2001 From: Ted Gould Date: Thu, 28 May 2009 10:20:23 -0500 Subject: Putting a little more error handling code in. --- src/indicator-messages.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/src/indicator-messages.c b/src/indicator-messages.c index bccffe7..efb52b2 100644 --- a/src/indicator-messages.c +++ b/src/indicator-messages.c @@ -65,7 +65,16 @@ get_menu (void) DBusGConnection * connection = dbus_g_bus_get(DBUS_BUS_SESSION, NULL); DBusGProxy * proxy = dbus_g_proxy_new_for_name(connection, DBUS_SERVICE_DBUS, DBUS_PATH_DBUS, DBUS_INTERFACE_DBUS); - org_freedesktop_DBus_start_service_by_name (proxy, INDICATOR_MESSAGES_DBUS_NAME, 0, &returnval, &error); + if (!org_freedesktop_DBus_start_service_by_name (proxy, INDICATOR_MESSAGES_DBUS_NAME, 0, &returnval, &error)) { + g_error("Unable to send message to DBus to start service: %s", error != NULL ? error->message : "(NULL error)" ); + g_error_free(error); + return NULL; + } + + if (returnval != DBUS_START_REPLY_SUCCESS && returnval != DBUS_START_REPLY_ALREADY_RUNNING) { + g_error("Return value isn't indicative of success: %d", returnval); + return NULL; + } return GTK_MENU(dbusmenu_gtkmenu_new(INDICATOR_MESSAGES_DBUS_NAME, INDICATOR_MESSAGES_DBUS_OBJECT)); } -- cgit v1.2.3 From 269238b32422367cc568d2c10e3683fec45734eb Mon Sep 17 00:00:00 2001 From: Ted Gould Date: Thu, 28 May 2009 10:29:41 -0500 Subject: Building a simple little activator to find out if we can activate the service. --- .bzrignore | 1 + test/indicator-messages-service-activate.build.sh | 3 ++ test/indicator-messages-service-activate.c | 53 +++++++++++++++++++++++ 3 files changed, 57 insertions(+) create mode 100644 test/indicator-messages-service-activate.build.sh create mode 100644 test/indicator-messages-service-activate.c diff --git a/.bzrignore b/.bzrignore index a3226e8..42bdaa3 100644 --- a/.bzrignore +++ b/.bzrignore @@ -18,3 +18,4 @@ libmessaging_la-im-menu-item.lo src/libmessaging_la-app-menu-item.lo data/indicator-messages.service indicator-messages-service +indicator-messages-service-activate diff --git a/test/indicator-messages-service-activate.build.sh b/test/indicator-messages-service-activate.build.sh new file mode 100644 index 0000000..87a0316 --- /dev/null +++ b/test/indicator-messages-service-activate.build.sh @@ -0,0 +1,3 @@ +#!/bin/sh + +gcc -o indicator-messages-service-activate indicator-messages-service-activate.c `pkg-config --cflags --libs dbus-1 dbus-glib-1` diff --git a/test/indicator-messages-service-activate.c b/test/indicator-messages-service-activate.c new file mode 100644 index 0000000..98c6522 --- /dev/null +++ b/test/indicator-messages-service-activate.c @@ -0,0 +1,53 @@ +/* +An indicator to show information that is in messaging applications +that the user is using. + +Copyright 2009 Canonical Ltd. + +Authors: + Ted Gould + +This program is free software: you can redistribute it and/or modify it +under the terms of the GNU General Public License version 3, as published +by the Free Software Foundation. + +This program is distributed in the hope that it will be useful, but +WITHOUT ANY WARRANTY; without even the implied warranties of +MERCHANTABILITY, SATISFACTORY QUALITY, 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, see . +*/ + +#include +#include +#include +#include "../src/dbus-data.h" + +int +main (int argc, char ** argv) +{ + g_type_init(); + + guint returnval = 0; + GError * error = NULL; + + DBusGConnection * connection = dbus_g_bus_get(DBUS_BUS_SESSION, NULL); + DBusGProxy * proxy = dbus_g_proxy_new_for_name(connection, DBUS_SERVICE_DBUS, DBUS_PATH_DBUS, DBUS_INTERFACE_DBUS); + + g_debug("Activating service: %s", INDICATOR_MESSAGES_DBUS_NAME); + if (!org_freedesktop_DBus_start_service_by_name (proxy, INDICATOR_MESSAGES_DBUS_NAME, 0, &returnval, &error)) { + g_error("Unable to send message to DBus to start service: %s", error != NULL ? error->message : "(NULL error)" ); + g_error_free(error); + return 1; + } + + if (returnval != DBUS_START_REPLY_SUCCESS && returnval != DBUS_START_REPLY_ALREADY_RUNNING) { + g_error("Return value isn't indicative of success: %d", returnval); + return 1; + } + + return 0; +} + -- cgit v1.2.3 From 7bd2ed28f7657191beb001f60109167ab1644a57 Mon Sep 17 00:00:00 2001 From: Ted Gould Date: Thu, 28 May 2009 10:33:26 -0500 Subject: Aw, bother, turns out I messed up the service file. Let's fix that. --- data/indicator-messages.service.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/data/indicator-messages.service.in b/data/indicator-messages.service.in index fbe8b4a..908079a 100644 --- a/data/indicator-messages.service.in +++ b/data/indicator-messages.service.in @@ -1,3 +1,3 @@ -[Indicator Messages] +[D-BUS Service] Name=com.ubuntu.indicator.messages Exec=@prefix@/bin/indicator-messages-service -- cgit v1.2.3