-
Notifications
You must be signed in to change notification settings - Fork 93
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release 2.16 #3148
Release 2.16 #3148
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3148 +/- ##
==========================================
+ Coverage 77.81% 77.82% +0.01%
==========================================
Files 1074 1064 -10
Lines 74893 74440 -453
==========================================
- Hits 58275 57933 -342
+ Misses 16618 16507 -111 ☔ View full report in Codecov by Sentry. |
Not all `ConsoleProvider`s will send a `removed` event on the `DeviceObserver` (in fact, only logind will), so we can't rely on that to clean up the handle in `device_watchers`. Furthermore, libinput itself will notice a device going away (by monitoring the fd) and generate `LIBINPUT_EVENT_DEVICE_REMOVED`, which we might process *before* the udev `REMOVED` event, and in *that* case, `device_removed` will have already deleted the `LibInputDevice` and so the udev event handler will *also* not clean up `device_watchers`. Resolve this by cleaning up `device_watchers` *only* in `device_removed`, and rely on the `DeviceObserver::removed` codepath triggering a libinput removal event. Fixes: #3149 (cherry picked from commit 5205713)
I've noticed that the changelog is missing at least on PR (#3145):
(I'm leaving this in a comment pending further checking) |
@AlanGriffiths we'll need this one cherry-picked: |
(cherry picked from commit cfcf5d0)
~rc5
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Woot!
Just waiting for things to build in the RC PPA:
https://launchpad.net/~mir-team/+archive/ubuntu/rc/+packages
~rc1
TestPlan