After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 751472 - Don't create libinput properties
Don't create libinput properties
Status: RESOLVED FIXED
Product: clutter
Classification: Platform
Component: x11
git master
Other Linux
: Normal normal
: ---
Assigned To: clutter-maint
clutter-maint
Depends on:
Blocks:
 
 
Reported: 2015-06-24 23:48 UTC by Peter Hutterer
Modified: 2015-07-07 15:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
0001-x11-don-t-create-the-libinput-tapping-property.patch (1.34 KB, patch)
2015-06-24 23:48 UTC, Peter Hutterer
accepted-commit_now Details | Review

Description Peter Hutterer 2015-06-24 23:48:10 UTC
Created attachment 306063 [details] [review]
0001-x11-don-t-create-the-libinput-tapping-property.patch

clutter checks for the libinput Tapping property, but in the process creates the property (not on the device, just in the global lookup table).

This confuses other layers of the stack that have similar checks - once mutter runs the property exists (not on the device, but in the global atom table).

Example for this confusion (though this one doesn't seem to be clutter's fault):
https://bugzilla.redhat.com/show_bug.cgi?id=1199825
Comment 1 Emmanuele Bassi (:ebassi) 2015-06-24 23:53:27 UTC
Review of attachment 306063 [details] [review]:

Pretty sure GDK does the same thing, since the code comes from a similar commit there:

  https://git.gnome.org/browse/gtk+/commit/?id=6f07d5e761f2c1d18825f73033ab7d095ea2c70d

Makes sense, though.
Comment 2 Emmanuele Bassi (:ebassi) 2015-06-24 23:54:37 UTC
Adding Carlos to the Cc list.
Comment 3 Peter Hutterer 2015-06-24 23:57:10 UTC
hah, thanks. I was trying to figure out where that last instance came from :)
Comment 4 Carlos Garnacho 2015-06-25 00:33:17 UTC
Doh, mea culpa for all of these. The patch looks good to me, if you don't get to doing a GTK+ one, I'll do tomorrow.
Comment 5 Peter Hutterer 2015-06-25 00:52:37 UTC
Carlos: I'll leave it to you for the GTK+ one, it's not as straightforward as this one and you'll know better what will break :)
Comment 6 Emmanuele Bassi (:ebassi) 2015-07-07 15:05:37 UTC
Pushed to master and to the clutter-1.22 branch, and it's included in the 1.22.4 stable release.

Thanks!