! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
xfce4-mouse-settings claims that XI is not present or too old
Status:
RESOLVED: WORKSFORME
Product:
Xfce4-settings
Component:
Mouse Settings

Comments

Description Christoph Mende 2009-02-12 20:02:42 CET
angelos@odin ~ % xfce4-mouse-settings 

(xfce4-mouse-settings:11678): xfce4-mouse-settings-CRITICAL **: XI is not present or too old.

In fact, it is too new. This is from the xfce4-settings build:
main.c:1226: warning: 'XGetExtensionVersion' is deprecated (declared at /usr/include/X11/extensions/XInput.h:1241)
and
pointers.c:138: warning: 'XGetExtensionVersion' is deprecated (declared at /usr/include/X11/extensions/XInput.h:1241)

This is not yet an issue for the average joe since my libXi is built from git (version 1.2.99.1 according to the xi.pc), but it will become one as soon as distros start shipping libXi 1.3 (Xorg 7.5 I guess)
Comment 1 Christoph Mende 2009-02-13 10:45:22 CET
Created attachment 2166 
fix xi check for xi 2.0

major_version.minor_version are 2.0 for libXi-1.3, so the logic in that check is broken here (minor_version is < 4), the above patch fixes that and uses the new XQueryInputVersion instead of XGetExtensionVersion, but it introduced a minor flaw.. the if evaluates to true with Xi 0.5-0.9, shouldn't be a problem though as the lowest version available seems to be 1.0
Comment 2 Christoph Mende 2009-02-13 10:49:48 CET
Created attachment 2167 
new patch

actually... here's a patch without said flaw
Comment 3 Christoph Mende 2009-02-13 10:52:09 CET
Created attachment 2168 
real new patch

eh, attached the old one before, sorry for the spam
Comment 4 Nick Schermer editbugs 2009-02-13 14:47:31 CET
Using XQueryInputVersion also enables the internal xi2 support, so i'm not sure this works with our code.
Comment 5 Nick Schermer editbugs 2009-02-13 15:14:15 CET
Created attachment 2170 
Check for 1.4 using XQueryInputVersion

How about this? Can you try if this works?
Comment 6 Brian J. Tarricone (not reading bugmail) 2009-02-13 17:22:41 CET
Not a 4.6 blocker since XI 2.x isn't out yet; scheduling for 4.6.1.
Comment 7 Christoph Mende 2009-02-14 00:43:02 CET
kind of works... the dialog starts up, but the devices list is empty, even when I set the major and minor in XQueryInputVersion to 0, 0 (which seems to be what's done internally when XGetExtensionVersion is used). anyway, my mouse is in the list with my patch
Comment 8 Brian J. Tarricone (not reading bugmail) 2009-02-14 00:50:38 CET
Is XI 2.0 backward compatible with XI 1.x?  If not, we shouldn't be advertising support for 2.0 unless we actually support it properly...
Comment 9 Brian J. Tarricone (not reading bugmail) 2009-02-14 00:52:08 CET
And no, this is *not* going into 4.6.0.  Only 4.6 blockers should go into trunk now.
Comment 10 Nick Schermer editbugs 2009-02-14 10:38:27 CET
(In reply to comment #8)
> Is XI 2.0 backward compatible with XI 1.x?  If not, we shouldn't be advertising
> support for 2.0 unless we actually support it properly...

No it's not, the returned data is not compatible, Therefore I tried XQueryInputVersion (xdisplay, 1, 4); which _should_ make X internally fallback to the old behaviour. But that probably doesn't work. Anyway mpx will most likely be added in xorg 7.6 (if it's not delayed again), so it's not a really important to fix, but it would be nice if it works and fallsback to the old behaviour.
Comment 11 Kevin Fenzi 2009-09-26 02:03:16 CEST
Looks like this is about to be an issue in Fedora 12 (not yet released). 

Fedora rawhide currently has libXi-1.2.99-11.20090825 and runs into this issue. ;( 

Any chance of a fix in a 4.6.2 or the like? :)
Comment 12 Christoph Mende 2011-04-09 20:57:38 CEST
I think this can be closed, I'm not getting this anymore with libXi-1.4.2 and xfce4-settings-4.8.
Any reason this is still open?

Bug #4929

Reported by:
Christoph Mende
Reported on: 2009-02-12
Last modified on: 2011-05-13

People

Assignee:
Nick Schermer
CC List:
3 users

Version

Attachments

fix xi check for xi 2.0 (1.56 KB, patch)
2009-02-13 10:45 CET , Christoph Mende
no flags
new patch (1.56 KB, patch)
2009-02-13 10:49 CET , Christoph Mende
no flags
real new patch (1.62 KB, patch)
2009-02-13 10:52 CET , Christoph Mende
no flags
Check for 1.4 using XQueryInputVersion (2.79 KB, patch)
2009-02-13 15:14 CET , Nick Schermer
no flags

Additional information