Fix a number of annoying issues in gbemol:

- sort albums for the selector
- fix crash if authentication failed
- make authentication actually work
- be consistent in section of the config file used
Bump revision

7 files changed, 213 insertions, 0 deletions

diff --git a/audio/gbemol/patches/patch-aa b/audio/gbemol/patches/patch-aa
new file mode 100644
index 00000000000..bdf794fd364
--- /dev/null
+++ b/audio/gbemol/patches/patch-aa
@@ -0,0 +1,43 @@
+$NetBSD: patch-aa,v 1.1 2009/09/03 01:29:34 joerg Exp $
+
+--- src/gbemol-app.c.orig	2009-04-25 04:38:51.000000000 +0200
++++ src/gbemol-app.c
+@@ -449,7 +449,7 @@ gbemol_app_init (GbemolApp* app)
+ 	gbemol_app_build_gui (app);
+ 
+ 	/* Check if there is a config file */
+-	if (!(host = gbemol_cfg_get_string ("MPD", "host")))
++	if (!(host = gbemol_cfg_get_string ("mpd", "host")))
+ 	{
+ 		app->mpd = gbemol_mpd_new_with_defaults ();
+ 
+@@ -471,13 +471,13 @@ gbemol_app_init (GbemolApp* app)
+ 	else 
+ 	{
+ 		/* We need a password? */
+-		if (gbemol_cfg_get_int ("MPD", "use_auth"))
+-			pass = gbemol_cfg_get_string ("MPD", "pass");
++		if (gbemol_cfg_get_int ("mpd", "use_auth"))
++			pass = gbemol_cfg_get_string ("mpd", "pass");
+ 		else
+ 			pass = NULL;
+ 
+-		app->mpd = gbemol_mpd_new (host, pass, gbemol_cfg_get_int ("MPD", "port"), 
+-				gbemol_cfg_get_double ("MPD", "timeout"));
++		app->mpd = gbemol_mpd_new (host, pass, gbemol_cfg_get_int ("mpd", "port"), 
++				gbemol_cfg_get_double ("mpd", "timeout"));
+ 		g_free (host);
+ 		g_free (pass);
+ 	}
+@@ -564,10 +564,7 @@ void gbemol_app_connect_mpd (GbemolApp* 
+ 	if (!app->mpd)
+ 		return;
+ 
+-	if (gbemol_cfg_get_bool("MPD", "use_auth"))
+-		gbemol_mpd_connect_and_authenticate (app->mpd);
+-	else
+-		gbemol_mpd_connect (app->mpd);
++	gbemol_mpd_connect (app->mpd);
+ 
+ 	gbemol_app_set_connected (app, gbemol_mpd_is_connected (app->mpd));
+ 	if (gbemol_mpd_is_connected (app->mpd))
diff --git a/audio/gbemol/patches/patch-ab b/audio/gbemol/patches/patch-ab
new file mode 100644
index 00000000000..b040fa36689
--- /dev/null
+++ b/audio/gbemol/patches/patch-ab
@@ -0,0 +1,37 @@
+$NetBSD: patch-ab,v 1.1 2009/09/03 01:29:34 joerg Exp $
+
+--- src/gbemol-library.c.orig	2009-04-25 04:38:51.000000000 +0200
++++ src/gbemol-library.c
+@@ -21,6 +21,7 @@
+ #include "gbemol-utils.h"
+ 
+ #include <gtk/gtk.h>
++#include <string.h>
+ 
+ static GObjectClass* parent_class = NULL;
+ 
+@@ -641,7 +642,6 @@ void gbemol_library_columns_dialog (Gbem
+ 	gtk_widget_destroy (dlg);
+ }
+ 
+-
+ /*
+  * Populates lst with the tags
+  */
+@@ -651,6 +651,8 @@ gbemol_library_populate_constraint (Gbem
+ 	GtkTreeIter iter_all, iter;	
+ 	GtkListStore* lst = lib->priv->lst_constraint[id];
+ 
++	tags = g_list_sort(tags, (GCompareFunc)strcmp);
++
+ 	gtk_list_store_clear (lst);
+ 	
+ 	gtk_list_store_append (lst, &iter_all);
+@@ -671,7 +673,6 @@ gbemol_library_populate_constraint (Gbem
+ 					-1);
+ 		} while ((tags = g_list_next (tags)));
+ 	}
+-
+ }
+ 
+ void 
diff --git a/audio/gbemol/patches/patch-ac b/audio/gbemol/patches/patch-ac
new file mode 100644
index 00000000000..40ded1a7209
--- /dev/null
+++ b/audio/gbemol/patches/patch-ac
@@ -0,0 +1,13 @@
+$NetBSD: patch-ac,v 1.1 2009/09/03 01:29:34 joerg Exp $
+
+--- src/gbemol-main.c.orig	2009-09-03 03:20:21.000000000 +0200
++++ src/gbemol-main.c
+@@ -107,7 +107,7 @@ int main (int argc, char *argv[])	
+ 	}
+ 
+ 	/* Update library on start */
+-	if (gbemol_mpd_is_connected (app->mpd) && gbemol_cfg_get_int ("MPD", "update-library"))
++	if (gbemol_mpd_is_connected (app->mpd) && gbemol_cfg_get_int ("mpd", "update-library"))
+ 		gbemol_mpd_database_update (app->mpd, "/");
+ 
+ 	gtk_main();
diff --git a/audio/gbemol/patches/patch-ad b/audio/gbemol/patches/patch-ad
new file mode 100644
index 00000000000..42ddcdaea29
--- /dev/null
+++ b/audio/gbemol/patches/patch-ad
@@ -0,0 +1,81 @@
+$NetBSD: patch-ad,v 1.1 2009/09/03 01:29:34 joerg Exp $
+
+--- src/gbemol-mpd.c.orig	2009-04-25 04:38:51.000000000 +0200
++++ src/gbemol-mpd.c
+@@ -2,6 +2,7 @@
+ 
+ #include "gbemol-mpd.h"
+ #include "gbemol-marshal.h"
++#include "gbemol-utils.h"
+ 
+ /* Properties */
+ enum {
+@@ -454,51 +455,31 @@ gboolean gbemol_mpd_connect (GbemolMpd* 
+ 	obj->priv->conn = mpd_newConnection (obj->priv->host, obj->priv->port, obj->priv->timeout);
+ 	
+ 	if (gbemol_mpd_finish_and_handle (obj))
+-		obj->priv->connected = FALSE;
+-	else
+-	{
+-		gbemol_mpd_get_not_commands_list (obj);
+-		if (obj->status)
+-			mpd_freeStatus (obj->status);
+-		obj->status = gbemol_mpd_get_status (obj);
+-		obj->priv->connected = TRUE;
+-		/* Emit a song-changed signal */
+-		g_signal_emit (obj, gbemol_mpd_signals [SONG_CHANGED_SIGNAL], 0, NULL);
+-	}
+-
+-	return obj->priv->connected;
+-}
+-
+-/* Connect */
+-gboolean gbemol_mpd_connect_and_authenticate (GbemolMpd* obj)
+-{
+-	if (obj->priv->conn)
+-		mpd_closeConnection (obj->priv->conn);
+-
+-	obj->priv->conn = mpd_newConnection (obj->priv->host, obj->priv->port, obj->priv->timeout);
+-	
+-	if (gbemol_mpd_finish_and_handle (obj))
+ 	{
+ 		obj->priv->connected = FALSE;
+ 		return FALSE;
+ 	}
+-	else
+-		obj->priv->connected = TRUE;
+ 
+-	/* Not authenticated, yet connected, so return TRUE, but mark as disconnected */
+-	if (!gbemol_mpd_authenticate (obj))
+-		obj->priv->connected = FALSE;
+-	else
++	if (gbemol_cfg_get_bool("mpd", "use_auth"))
+ 	{
+-		gbemol_mpd_get_not_commands_list (obj);
+-		if (obj->status)
+-			mpd_freeStatus (obj->status);
+-		obj->status = gbemol_mpd_get_status (obj);
+-		/* Emit a song-changed signal */
+-		g_signal_emit (obj, gbemol_mpd_signals [SONG_CHANGED_SIGNAL], 0, NULL);
++		if (!obj->priv->pass)
++			obj->priv->pass = gbemol_cfg_get_string("mpd", "pass");
++		if (!obj->priv->pass || !gbemol_mpd_authenticate (obj))
++		{
++			obj->priv->connected = FALSE;
++			return FALSE;		
++		}
+ 	}
+ 
+-	return TRUE;
++	gbemol_mpd_get_not_commands_list (obj);
++	if (obj->status)
++		mpd_freeStatus (obj->status);
++	obj->status = gbemol_mpd_get_status (obj);
++	obj->priv->connected = TRUE;
++	/* Emit a song-changed signal */
++	g_signal_emit (obj, gbemol_mpd_signals [SONG_CHANGED_SIGNAL], 0, NULL);
++
++	return obj->priv->connected;
+ }
+ 
+ void gbemol_mpd_disconnect (GbemolMpd *obj)
diff --git a/audio/gbemol/patches/patch-ae b/audio/gbemol/patches/patch-ae
new file mode 100644
index 00000000000..d8227983dee
--- /dev/null
+++ b/audio/gbemol/patches/patch-ae
@@ -0,0 +1,12 @@
+$NetBSD: patch-ae,v 1.1 2009/09/03 01:29:34 joerg Exp $
+
+--- src/gbemol-mpd.h.orig	2009-04-25 04:38:51.000000000 +0200
++++ src/gbemol-mpd.h
+@@ -47,7 +47,6 @@ GbemolMpd* gbemol_mpd_new (const gchar *
+ 
+ /* Connection Methods */
+ gboolean gbemol_mpd_connect (GbemolMpd *obj);
+-gboolean gbemol_mpd_connect_and_authenticate (GbemolMpd* obj);
+ void gbemol_mpd_disconnect (GbemolMpd *obj);
+ gboolean gbemol_mpd_is_connected (GbemolMpd *obj);
+ gchar* gbemol_mpd_get_version (GbemolMpd* obj);
diff --git a/audio/gbemol/patches/patch-af b/audio/gbemol/patches/patch-af
new file mode 100644
index 00000000000..4eded5244b9
--- /dev/null
+++ b/audio/gbemol/patches/patch-af
@@ -0,0 +1,14 @@
+$NetBSD: patch-af,v 1.1 2009/09/03 01:29:34 joerg Exp $
+
+--- src/gbemol-preferences.c.orig	2009-04-25 04:38:51.000000000 +0200
++++ src/gbemol-preferences.c
+@@ -522,6 +522,9 @@ gbemol_preferences_populate_outputs (Gbe
+ 		return;
+ 	l = gbemol_mpd_output_get_list (obj->app->mpd);
+ 
++	if (!l)
++		return;
++
+ 	do
+ 	{
+ 		out = l->data;
diff --git a/audio/gbemol/patches/patch-ag b/audio/gbemol/patches/patch-ag
new file mode 100644
index 00000000000..4c5984048b6
--- /dev/null
+++ b/audio/gbemol/patches/patch-ag
@@ -0,0 +1,13 @@
+$NetBSD: patch-ag,v 1.1 2009/09/03 01:29:34 joerg Exp $
+
+--- src/gbemol-utils.c.orig	2009-04-25 04:38:51.000000000 +0200
++++ src/gbemol-utils.c
+@@ -374,7 +374,7 @@ GdkPixbuf* gbemol_get_song_image_from_id
+ 	gulong size = 0;
+ 	guchar  *data = NULL;
+ 
+-	if (!(songs_path = gbemol_cfg_get_string ("MPD", "music_dir")))
++	if (!(songs_path = gbemol_cfg_get_string ("mpd", "music_dir")))
+ 		return NULL;
+ 
+ 	path = g_strdup_printf ("%s/%s", songs_path, song->file);