Merge lp:~bregma/geis/lp-903320 into lp:geis
Status: | Merged |
---|---|
Merged at revision: | 181 |
Proposed branch: | lp:~bregma/geis/lp-903320 |
Merge into: | lp:geis |
Diff against target: |
48 lines (+11/-6) 2 files modified
configure.ac (+1/-1) tools/geisview/geisview (+10/-5) |
To merge this branch: | bzr merge lp:~bregma/geis/lp-903320 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Chase Douglas (community) | 2011-12-12 | Approve on 2011-12-12 | |
Review via email:
|
Description of the change
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) wrote : | # |
Stephen M. Webb (bregma) wrote : | # |
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) wrote : | # |
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.
Is the revision bump supposed to be before the fix? If it's a previous revision, how did we release with an older revision?