894382 - [Gamepad] Support libudev.so.1

Status: VERIFIED FIXED

(Core :: Hardware Abstraction Layer (HAL), defect)

Description

[(not currently active) Ted Mielczarek]

Currently if you don't have libudev installed, the Linux Gamepad backend just fails to find any Gamepad devices. This kind of sucks since apparently it's not installed by default on Ubuntu (and maybe other distros). We can fall back to a much crappier "scan the filesystem for /dev/input/js*" code path if we don't have libudev, which would be better than nothing.

Comment 1

[(not currently active) Ted Mielczarek]

After installing a fresh Ubuntu 13.04, I found that libudev was installed, so maybe that's a red herring. I did notice (after Chris Coulson pointed it out), that the binary version has been bumped, so it's libudev.so.1, and my code only looks for libudev.so.0. This would result in the same behavior, but it's much simpler to fix.

Comment 2

[(not currently active) Ted Mielczarek]

This indeed appears to be the case. It looks like the ABI change didn't actually change anything we're using:
https://lists.fedoraproject.org/pipermail/devel/2012-June/168227.html

so it should be okay to just try to load both libraries in turn. I have a patch for this, I'll spin a try build so we can test.

Comment 3

[(not currently active) Ted Mielczarek]

Attachment: support libudev.so.1 in Linux Gamepad backend

This seems to work fine on my Ubuntu 12.04 system (which has libudev.so.0). I'll try it on my 13.04 VM as well. I've also pushed it to Try as a sanity check, and to get some builds that QA can use to test:
https://tbpl.mozilla.org/?tree=Try&rev=758fa557e127

Comment 4

[Ada [:adalucinet]]

Mozilla/5.0 (X11; Linux i686; rv:24.0) Gecko/20130718 Firefox/24.0

It works just fine with the try build (20130718083050) on Ubuntu 13.04 x32.

Comment 5

[Ada [:adalucinet]]

Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20130718 Firefox/24.0

WFM also on Ubuntu 12.10 x64 (with libudev.so.0).

Comment 6

[(not currently active) Ted Mielczarek]

https://hg.mozilla.org/integration/mozilla-inbound/rev/12920f3feadc

Comment 7

[(not currently active) Ted Mielczarek]

Alexandra: thanks for the testing!

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/12920f3feadc

Comment 9

[Ada [:adalucinet]]

Marking as verified fixed based on comment 4 and comment 5.