releasing version 0.2.15-0ubuntu1

5 files changed, 139 insertions, 10 deletions

diff --git a/ChangeLog b/ChangeLog
index b70f74d..4a8d8a8 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,62 @@
 # Generated by Makefile. Do not edit.
 
+2011-03-10  Ted Gould  <ted@gould.cx>
+
+	0.2.15
+
+2011-03-10  Ted Gould  <ted@gould.cx>
+
+	Use the --no-lock parameter
+
+2011-03-10  Ted Gould  <ted@gould.cx>
+
+	Add the --no-lock option to activate the session.
+
+2011-03-10  Ted Gould  <ted@gould.cx>
+
+	Looking in a directory for extra launchers
+
+2011-03-10  Ted Gould  <ted@gould.cx>
+
+	Embed the function so it's not one separator ever, but once per run
+
+2011-03-10  Ted Gould  <ted@gould.cx>
+
+	Add some cleanup and some comments
+
+2011-03-10  Ted Gould  <ted@gould.cx>
+
+	Sort the launchers
+
+2011-03-10  Ted Gould  <ted@gould.cx>
+
+	Reducing the scope of some of our variables
+
+2011-03-10  Ted Gould  <ted@gould.cx>
+
+	Closing the directory after we're done with it
+
+2011-03-10  Ted Gould  <ted@gould.cx>
+
+	Putting the appinfo's into a list so that we can sort them
+
+2011-03-10  Martin Pitt  <martin.pitt@ubuntu.com>
+
+	Drop support for indicator-session-extra.desktop
+
+2011-03-10  Martin Pitt  <martin.pitt@canonical.com>
+
+	Add support for extra launcher dir
+	
+	Hardcoding a single extra launcher path is impractical for distributions, as
+	multiple packages would collide on the name and wouldn't give any extra
+	customization possibility for OEMs.
+	
+	Obsolete /usr/share/applications/indicator-session-extra.desktop and instead
+	read /usr/share/indicators/session/applications/*.desktop
+	
+	https://launchpad.net/bugs/727823
+
 2011-02-17  Ted Gould  <ted@gould.cx>
 
 	0.2.14
diff --git a/configure b/configure
index 0cd412f..e65cc7d 100755
--- a/configure
+++ b/configure
@@ -2882,7 +2882,7 @@ fi
 
 # Define the identity of the package.
  PACKAGE=indicator-session
- VERSION=0.2.14
+ VERSION=0.2.15
 
 
 cat >>confdefs.h <<_ACEOF
diff --git a/configure.ac b/configure.ac
index e12c744..724831f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -4,7 +4,7 @@ AC_INIT(src/indicator-session.c)
 AC_PREREQ(2.53)
 
 AM_CONFIG_HEADER(config.h)
-AM_INIT_AUTOMAKE(indicator-session, 0.2.14)
+AM_INIT_AUTOMAKE(indicator-session, 0.2.15)
 
 AM_MAINTAINER_MODE
 
diff --git a/debian/changelog b/debian/changelog
index a7442e5..f7694ef 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,13 @@
+indicator-session (0.2.15-0ubuntu1) natty; urgency=low
+
+  * New upstream release.
+    ∘ Support adding arbitrary items to the end of the session
+      menu. (LP: #727823)
+    ∘ Call guest session with --no-lock to ensure there is no
+      double locking (LP: #636693)
+
+ -- Ted Gould <ted@ubuntu.com>  Thu, 10 Mar 2011 16:10:19 -0600
+
 indicator-session (0.2.14-0ubuntu1) natty; urgency=low
 
   * New upstream release.
diff --git a/src/session-service.c b/src/session-service.c
index 1d8cd3a..dc53f74 100644
--- a/src/session-service.c
+++ b/src/session-service.c
@@ -54,7 +54,7 @@ with this program.  If not, see <http://www.gnu.org/licenses/>.
 #define UP_OBJECT     "/org/freedesktop/UPower"
 #define UP_INTERFACE  "org.freedesktop.UPower"
 
-#define DESKTOP_FILE  "/usr/share/applications/indicator-session-extra.desktop"
+#define EXTRA_LAUNCHER_DIR "/usr/share/indicators/session/applications"
 
 #define GUEST_SESSION_LAUNCHER  "/usr/share/gdm/guest-session/guest-session-launch"
 
@@ -424,7 +424,7 @@ activate_guest_session (DbusmenuMenuitem * mi, guint timestamp, gpointer user_da
 		g_warning("Unable to activate guest session, falling back to command line activation.");
 	}
 
-	if (!g_spawn_command_line_async(GUEST_SESSION_LAUNCHER, &error)) {
+	if (!g_spawn_command_line_async(GUEST_SESSION_LAUNCHER " --no-lock", &error)) {
 		g_warning("Unable to start guest session: %s", error->message);
 		g_error_free(error);
 	}
@@ -517,6 +517,28 @@ desktop_activate_cb (DbusmenuMenuitem * mi, guint timestamp, gpointer data)
 	return;
 }
 
+/* Look at the GAppInfo structures and sort based on
+   the application names */
+static gint
+sort_app_infos (gconstpointer a, gconstpointer b)
+{
+	GAppInfo * appa = G_APP_INFO(a);
+	GAppInfo * appb = G_APP_INFO(b);
+
+	const gchar * namea = NULL;
+	const gchar * nameb = NULL;
+
+	if (appa != NULL) {
+		namea = g_app_info_get_name(appa);
+	}
+
+	if (appb != NULL) {
+		nameb = g_app_info_get_name(appb);
+	}
+
+	return g_strcmp0(namea, nameb);
+}
+
 /* Builds up the menu for us */
 static void
 rebuild_items (DbusmenuMenuitem *root,
@@ -713,19 +735,59 @@ rebuild_items (DbusmenuMenuitem *root,
 
 	update_menu_entries(restart_shutdown_logout_mi);
 
-	if (g_file_test(DESKTOP_FILE, G_FILE_TEST_EXISTS)) {
-		GAppInfo * appinfo = G_APP_INFO(g_desktop_app_info_new_from_filename(DESKTOP_FILE));
+	/* now add extra launchers */
+	GDir *extra_launchers_dir;
+	extra_launchers_dir = g_dir_open (EXTRA_LAUNCHER_DIR, 0, NULL);
+	if (extra_launchers_dir != NULL) {
+		GList * launchers = NULL;
 
-		if (appinfo != NULL) {
-			DbusmenuMenuitem * separator = dbusmenu_menuitem_new();
-			dbusmenu_menuitem_property_set(separator, DBUSMENU_MENUITEM_PROP_TYPE, DBUSMENU_CLIENT_TYPES_SEPARATOR);
-			dbusmenu_menuitem_child_append(root, separator);
+		/* Find all the desktop files we want to use */
+		for (;;) {
+			const gchar *extra_launcher_file;
 
+			extra_launcher_file = g_dir_read_name (extra_launchers_dir);
+			if (extra_launcher_file == NULL)
+				break;
+			if (!g_str_has_suffix (extra_launcher_file, ".desktop"))
+				continue;
+
+			gchar *full_path = g_build_filename (EXTRA_LAUNCHER_DIR, extra_launcher_file, NULL);
+			GAppInfo * appinfo = G_APP_INFO(g_desktop_app_info_new_from_filename (full_path));
+			g_free (full_path);
+
+			launchers = g_list_prepend(launchers, appinfo);
+		}
+		g_dir_close(extra_launchers_dir);
+
+		/* Sort the desktop files based on their names */
+		launchers = g_list_sort(launchers, sort_app_infos);
+
+		/* Turn each one into a separate menu item */
+		GList * launcher = NULL;
+		gboolean sepadded = FALSE;
+		for (launcher = launchers; launcher != NULL; launcher = g_list_next(launcher)) {
+			GAppInfo * appinfo = G_APP_INFO(launcher->data);
+
+			/* Make sure we have a separator */
+			if (!sepadded) {
+				DbusmenuMenuitem * separator = dbusmenu_menuitem_new();
+				dbusmenu_menuitem_property_set(separator, DBUSMENU_MENUITEM_PROP_TYPE, DBUSMENU_CLIENT_TYPES_SEPARATOR);
+				dbusmenu_menuitem_child_append(root, separator);
+				g_object_unref(separator);
+				sepadded = TRUE;
+			}
+
+			/* Build the item */
 			DbusmenuMenuitem * desktop_mi = dbusmenu_menuitem_new();
 			dbusmenu_menuitem_property_set(desktop_mi, DBUSMENU_MENUITEM_PROP_LABEL, g_app_info_get_name(appinfo));
 			g_signal_connect(G_OBJECT(desktop_mi), DBUSMENU_MENUITEM_SIGNAL_ITEM_ACTIVATED, G_CALLBACK(desktop_activate_cb), appinfo);
+			g_object_weak_ref(G_OBJECT(desktop_mi), (GWeakNotify)g_object_unref, appinfo);
+
+			/* Put into the menu */
 			dbusmenu_menuitem_child_append(root, desktop_mi);
 		}
+
+		g_list_free(launchers);
 	}
 
 	return;