Merge lp:~bregma/geis/lp-903320 into lp:geis

[Stephen M. Webb]

Allows geisview to fall back to the XDB back end gracefully if no back end is explicitly requested on the command line.

No test case is required, running geisview with no back end specified and no geis daemon running (the current default situation) yielded no results, geisview with this patch yields results.

[Chase Douglas (chasedouglas)]

Is the revision bump supposed to be before the fix? If it's a previous revision, how did we release with an older revision?

[Stephen M. Webb (bregma)]

On 12/12/2011 03:06 PM, Chase Douglas wrote:
> Is the revision bump supposed to be before the fix? If it's a previous revision, how did we release with an older revision?

I must have forgotten to push up the version bump to lp:utouch-geis when
I released 2.2.2, it should not be a part of this merge request. I've
been using git too much lately, it tells you when your local repo is
ahead of a remote one.

I've pushed the version bump up now, so that part can be ignored for the
purposes of this merge request.

[Chase Douglas (chasedouglas)]

Looks sane to me. If I am reading it correctly, instead of assuming the DBUS backend by default it now does not specify a backend. This allows the XCB fallback when the DBUS backend is unavailable.

review: Approve

=== modified file 'configure.ac'
--- configure.ac	2011-10-22 03:50:55 +0000
+++ configure.ac	2011-12-12 19:35:32 +0000
@@ -21,7 +21,7 @@
 
 AC_PREREQ(2.65)
 AC_INIT([utouch-geis],
-	[2.2.1],
+	[2.2.2],
 	[[e-mail]],,
 	[https://launchpad.net/utouch-geis])
 AC_CONFIG_HEADERS([geis_config.h])

=== modified file 'tools/geisview/geisview'
--- tools/geisview/geisview	2011-10-18 20:43:58 +0000
+++ tools/geisview/geisview	2011-12-12 19:35:32 +0000
@@ -54,15 +54,20 @@
         self._builder.add_from_file(os.path.join(geisview.defaults.ui_dir,
                                                  "geisview.ui"))
         self._builder.connect_signals(self)
+        self._args = args;
 
         self._main_window = self._builder.get_object("main_window")
         self._filters_dialog = self._builder.get_object("filters_dialog")
 
-        be = _geis_backends.get(args.backend, geis.GEIS_INIT_UTOUCH_DBUS_BACKEND)
+        be = _geis_backends.get(args.backend, None)
         try:
-            self._geis = geis.Geis(geis.GEIS_INIT_TRACK_DEVICES,
-                                   geis.GEIS_INIT_TRACK_GESTURE_CLASSES,
-                                   be)
+            if be:
+                self._geis = geis.Geis(geis.GEIS_INIT_TRACK_DEVICES,
+                                       geis.GEIS_INIT_TRACK_GESTURE_CLASSES,
+                                       be)
+            else:
+                self._geis = geis.Geis(geis.GEIS_INIT_TRACK_DEVICES,
+                                       geis.GEIS_INIT_TRACK_GESTURE_CLASSES)
         except Exception as ex:
             box = gtk.MessageDialog(None,
                                     gtk.DIALOG_MODAL,
@@ -281,7 +286,7 @@
         self.add_argument('-w', '--window',  action='store', dest='windowid',
                           help=_('indicates the X windowid to receive input'))
         self.add_argument('-b', '--backend', action='store', dest='backend',
-                          choices=_geis_backends.keys(),
+                          choices=_geis_backends.keys(), default=None,
                           help=_('selects the recognizer back end'))