Merge lp:~bregma/evemu/lp-1152774 into lp:evemu

[Stephen M. Webb]

Fix python bindings crash (lp: #1152774).

= Problem description =

Using the evemu python bindings results in a crash unless the libevemu-dev package is installed.

= The fix =

The python bindings need to use the proper search string for the DSO.

= Test coverage =

It is not possible to provide an automated test since this situation does not occur in the build environment.

[Mathieu Trudel-Lapierre (cyphermox)]

We should double check that the libevemu-dev package also gets in the depends for the python bindings, in case this isn't explicitly what you're fixing here.

[Stephen M. Webb (bregma)]

The point is that the python-evemu runtime binary package should _not_ require the libevemu-dev development binary package in order to run without crashing. Adding a package dependency is wrong; fixing the DSO search string in the python bindings is the only change required.

[Mathieu Trudel-Lapierre (cyphermox)]

Please also add the Depends to python-evemu in debian/control.

review: Needs Fixing

[Mathieu Trudel-Lapierre (cyphermox)]

Disregard my message. Still you should consider having it in Suggest/Recommends depending on how important it is to get some of the features of evemu.

Consider this: how else are people expected to get the DSO installed? Is it depended on elsewhere? In which case you can ask whether a package that build-depends on both really should, or if it's python-evemu's job to pull it if it's needed.

review: Needs Information

[Mathieu Trudel-Lapierre (cyphermox)]

Could you at least add an integration test or autopkgtests to make sure the package works with the changes? (and verifying the fix)

I care little about the DSO/depends change itself, just make sure people get the right package they need to do whatever they need to do. If they may need to have the shared lib installed, then we should provide a way that it's clear the lib may be needed.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:68
http://jenkins.qa.ubuntu.com/job/evemu-ci/1/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/evemu-raring-amd64-ci/1//console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/evemu-raring-armhf-ci/1//console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/evemu-raring-i386-ci/1//console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/evemu-ci/1//rebuild/?

review: Approve (continuous-integration)

Unmerged revisions

68. By Stephen M. Webb on 2013-03-08

Fix crash when using python bindings without the -dev package installed (lp: #1152774).

=== modified file 'python/evemu/const.py'
--- python/evemu/const.py	2012-07-05 17:12:59 +0000
+++ python/evemu/const.py	2013-03-08 22:36:20 +0000
@@ -1,4 +1,4 @@
-LIB = "libevemu.so"
+LIB = "evemu"
 DEFAULT_LIB = "/usr/lib/libevemu.so"
 LOCAL_LIB = "../src/.libs/libevemu.so"
 UINPUT_NODE = "/dev/uinput"