Merge lp:~dbusmenu-team/libdbusmenu/ubuntu into lp:~ubuntu-desktop/libdbusmenu/ubuntu

[Ted Gould]

0.3.11

[lp:~dbusmenu-team/libdbusmenu/ubuntu]

90. By Ken VanDine on 2010-08-19

* New upstream release.
  * Fix to handle whitespace in layout XML better
  * Fixes to test scripts on ARM

=== modified file 'configure'
--- configure	2010-08-12 16:22:27 +0000
+++ configure	2010-08-19 18:52:42 +0000
@@ -1,6 +1,6 @@
 #! /bin/sh
 # Guess values for system-dependent variables and create Makefiles.
-# Generated by GNU Autoconf 2.67 for libdbusmenu 0.3.10.
+# Generated by GNU Autoconf 2.67 for libdbusmenu 0.3.11.
 #
 # Report bugs to <[e-mail]>.
 #
@@ -703,8 +703,8 @@
 # Identity of this package.
 PACKAGE_NAME='libdbusmenu'
 PACKAGE_TARNAME='libdbusmenu'
-PACKAGE_VERSION='0.3.10'
-PACKAGE_STRING='libdbusmenu 0.3.10'
+PACKAGE_VERSION='0.3.11'
+PACKAGE_STRING='libdbusmenu 0.3.11'
 PACKAGE_BUGREPORT='[e-mail]'
 PACKAGE_URL=''
 
@@ -1533,7 +1533,7 @@
   # Omit some internal or obsolete options to make the list less imposing.
   # This message is too long to be a string in the A/UX 3.1 sh.
   cat <<_ACEOF
-\`configure' configures libdbusmenu 0.3.10 to adapt to many kinds of systems.
+\`configure' configures libdbusmenu 0.3.11 to adapt to many kinds of systems.
 
 Usage: $0 [OPTION]... [VAR=VALUE]...
 
@@ -1603,7 +1603,7 @@
 
 if test -n "$ac_init_help"; then
   case $ac_init_help in
-     short | recursive ) echo "Configuration of libdbusmenu 0.3.10:";;
+     short | recursive ) echo "Configuration of libdbusmenu 0.3.11:";;
    esac
   cat <<\_ACEOF
 
@@ -1741,7 +1741,7 @@
 test -n "$ac_init_help" && exit $ac_status
 if $ac_init_version; then
   cat <<\_ACEOF
-libdbusmenu configure 0.3.10
+libdbusmenu configure 0.3.11
 generated by GNU Autoconf 2.67
 
 Copyright (C) 2010 Free Software Foundation, Inc.
@@ -2112,7 +2112,7 @@
 This file contains any messages produced by compilers while
 running configure, to aid debugging if configure makes a mistake.
 
-It was created by libdbusmenu $as_me 0.3.10, which was
+It was created by libdbusmenu $as_me 0.3.11, which was
 generated by GNU Autoconf 2.67.  Invocation command line was
 
   $ $0 $@
@@ -2933,7 +2933,7 @@
 
 # Define the identity of the package.
  PACKAGE=libdbusmenu
- VERSION=0.3.10
+ VERSION=0.3.11
 
 
 # Some tools Automake needs.
@@ -13065,7 +13065,7 @@
 ###########################
 
 LIBDBUSMENU_CURRENT=1
-LIBDBUSMENU_REVISION=12
+LIBDBUSMENU_REVISION=13
 LIBDBUSMENU_AGE=0
 
 
@@ -14379,7 +14379,7 @@
 # report actual input values of CONFIG_FILES etc. instead of their
 # values after options handling.
 ac_log="
-This file was extended by libdbusmenu $as_me 0.3.10, which was
+This file was extended by libdbusmenu $as_me 0.3.11, which was
 generated by GNU Autoconf 2.67.  Invocation command line was
 
   CONFIG_FILES    = $CONFIG_FILES
@@ -14445,7 +14445,7 @@
 cat >>$CONFIG_STATUS <<_ACEOF || ac_write_fail=1
 ac_cs_config="`$as_echo "$ac_configure_args" | sed 's/^ //; s/[\\""\`\$]/\\\\&/g'`"
 ac_cs_version="\\
-libdbusmenu config.status 0.3.10
+libdbusmenu config.status 0.3.11
 configured by $0, generated by GNU Autoconf 2.67,
   with options \\"\$ac_cs_config\\"
 
 
=== modified file 'configure.ac'
--- configure.ac	2010-08-12 16:22:27 +0000
+++ configure.ac	2010-08-19 18:52:42 +0000
@@ -1,11 +1,11 @@
 
-AC_INIT(libdbusmenu, 0.3.10, [e-mail])
+AC_INIT(libdbusmenu, 0.3.11, [e-mail])
 AC_COPYRIGHT([Copyright 2009,2010 Canonical])
 
 AC_PREREQ(2.62)
 
 AM_CONFIG_HEADER(config.h)
-AM_INIT_AUTOMAKE(libdbusmenu, 0.3.10, [-Wno-portability])
+AM_INIT_AUTOMAKE(libdbusmenu, 0.3.11, [-Wno-portability])
 
 AM_MAINTAINER_MODE
 
@@ -99,7 +99,7 @@
 ###########################
 
 LIBDBUSMENU_CURRENT=1
-LIBDBUSMENU_REVISION=12
+LIBDBUSMENU_REVISION=13
 LIBDBUSMENU_AGE=0
 
 AC_SUBST(LIBDBUSMENU_CURRENT)
 
=== modified file 'debian/changelog'
--- debian/changelog	2010-08-12 19:21:26 +0000
+++ debian/changelog	2010-08-19 18:52:42 +0000
@@ -1,3 +1,11 @@
+libdbusmenu (0.3.11-0ubuntu1~ppa1) maverick; urgency=low
+
+  * New upstream release.
+    * Fix to handle whitespace in layout XML better
+    * Fixes to test scripts on ARM
+
+ -- Ted Gould <[e-mail]>  Thu, 19 Aug 2010 13:47:33 -0500
+
 libdbusmenu (0.3.10-0ubuntu1) maverick; urgency=low
 
   * New upstream release.
 
=== modified file 'debian/control'
--- debian/control	2010-08-12 19:21:26 +0000
+++ debian/control	2010-08-19 18:52:42 +0000
@@ -20,7 +20,10 @@
                gir1.0-glib-2.0,
                gir1.0-gtk-2.0,
                libx11-dev (>= 1.3), 
-               valac
+               valac,
+#  Check dependencies
+               xvfb,
+               dbus-test-runner
 Standards-Version: 3.9.1
 Homepage: https://launchpad.net/dbusmenu
 Vcs-Bzr: https://code.launchpad.net/~dbusmenu-team/dbusmenu/ubuntu
 
=== modified file 'debian/rules'
--- debian/rules	2010-08-12 19:21:26 +0000
+++ debian/rules	2010-08-19 18:52:42 +0000
@@ -3,6 +3,7 @@
 include /usr/share/cdbs/1/rules/debhelper.mk
 include /usr/share/cdbs/1/class/gnome.mk
 
+DEB_MAKE_CHECK_TARGET := check
 DEB_CONFIGURE_EXTRA_FLAGS += --disable-scrollkeeper --enable-gtk-doc
 LDFLAGS += -Wl,-z,defs -Wl,--as-needed
 
 
=== modified file 'docs/libdbusmenu-glib/reference/version.xml'
--- docs/libdbusmenu-glib/reference/version.xml	2010-08-12 16:22:27 +0000
+++ docs/libdbusmenu-glib/reference/version.xml	2010-08-19 18:52:42 +0000
@@ -1,1 +1,1 @@
-0.3.10
+0.3.11
 
=== modified file 'docs/libdbusmenu-gtk/reference/version.xml'
--- docs/libdbusmenu-gtk/reference/version.xml	2010-08-12 16:22:27 +0000
+++ docs/libdbusmenu-gtk/reference/version.xml	2010-08-19 18:52:42 +0000
@@ -1,1 +1,1 @@
-0.3.10
+0.3.11
 
=== modified file 'libdbusmenu-glib/client.c'
--- libdbusmenu-glib/client.c	2010-07-20 20:22:40 +0000
+++ libdbusmenu-glib/client.c	2010-08-19 18:52:42 +0000
@@ -1126,6 +1126,9 @@
 		/* g_debug("Looking at child: %d", position); */
 		gint childid = parse_node_get_id(children);
 		if (childid < 0) {
+			/* Don't increment the position when there isn't a valid
+			   node in the XML tree.  It's probably a comment. */
+			position--;
 			continue;
 		}
 		DbusmenuMenuitem * childmi = NULL;
@@ -1143,11 +1146,17 @@
 		}
 
 		if (childmi == NULL) {
+			#ifdef MASSIVEDEBUGGING
+			g_debug("Building new menu item %d at position %d", childid, position);
+			#endif
 			/* If we can't recycle, then we build a new one */
 			childmi = parse_layout_new_child(childid, client, item);
 			dbusmenu_menuitem_child_add_position(item, childmi, position);
 			g_object_unref(childmi);
 		} else {
+			#ifdef MASSIVEDEBUGGING
+			g_debug("Recycling menu item %d at position %d", childid, position);
+			#endif
 			/* If we can recycle, make sure it's in the right place */
 			dbusmenu_menuitem_child_reorder(item, childmi, position);
 			parse_layout_update(childmi, client);
@@ -1175,6 +1184,19 @@
 	children = node->children;
 	GList * childmis = dbusmenu_menuitem_get_children(item);
 	while (children != NULL && childmis != NULL) {
+		gint xmlid = parse_node_get_id(children);
+		/* If this isn't a valid menu item we need to move on
+		   until we have one.  This avoids things like comments. */
+		if (xmlid < 0) {
+			children = children->next;
+			continue;
+		}
+
+		#ifdef MASSIVEDEBUGGING
+		gint miid = dbusmenu_menuitem_get_id(DBUSMENU_MENUITEM(childmis->data));
+		g_debug("Recursing parse_layout_xml.  XML ID: %d  MI ID: %d", xmlid, miid);
+		#endif
+		
 		parse_layout_xml(client, children, DBUSMENU_MENUITEM(childmis->data), item, proxy);
 
 		children = children->next;
 
=== modified file 'tests/test-glib-properties-client.c'
--- tests/test-glib-properties-client.c	2010-02-05 18:10:47 +0000
+++ tests/test-glib-properties-client.c	2010-08-19 18:52:42 +0000
@@ -121,7 +121,7 @@
 layout_updated (DbusmenuClient * client, gpointer data)
 {
 	g_debug("Layout Updated");
-	g_timeout_add (250, layout_verify_timer, client);
+	g_timeout_add (500, layout_verify_timer, client);
 	return;
 }