Merge lp:~robertcarr/mir/1233944-addendum into lp:~mir-team/mir/trunk
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | kevin gunn | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 1089 | ||||
Proposed branch: | lp:~robertcarr/mir/1233944-addendum | ||||
Merge into: | lp:~mir-team/mir/trunk | ||||
Diff against target: |
65 lines (+24/-1) 2 files modified
3rd_party/android-input/android/frameworks/base/services/input/EventHub.cpp (+23/-1) 3rd_party/android-input/android/frameworks/base/services/input/EventHub.h (+1/-0) |
||||
To merge this branch: | bzr merge lp:~robertcarr/mir/1233944-addendum | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Robert Ancell | Approve | ||
Review via email: mp+188983@code.launchpad.net |
Commit message
Fix for LP#1233944
Fixes the Mir-side of bug https:/
Event files are first created with root:root permissions and only later
udev rules are applied to it, changing its permissions to root:android-input
and therefore making it readable by unity8-mir
in short: Retry opening a file when its permissions change as it might be
readable now.
Description of the change
Reproposing from my addendum branch as we want to land this quickly and dandrader is EOD. Thanks dandrader :D
==
Fixes bug 1233944.
But further work might be required (likely outside mir) due to the following error message when android-input loads the autopilot-finger device file:
"""
[InputReader] Touch device 'autopilot-finger' did not report support for X or Y axis! The device will be inoperable.
"""
this change is already approved on lp:~mir-team/mir/development-branch
proposing to trunk to fast track outside of dev branch merger