| wgrant | tjaalton: Are input-device properties meant to actually work on Intrepid? I can see that you've patched xinput and libxi to support them, but xinput can't seem to talk properly. | 03:43 |
|---|---|---|
| wgrant | (upstream xinput seems to want XI2 support to do anything with input properties... does the server only return them if it detects that the client knows about XI2?) | 03:45 |
| tjaalton | wgrant: it should work, the properties stuff has been backported to XI 1.5 | 07:31 |
| tjaalton | but I have to double-check it when I have more bandwith | 07:31 |
| tjaalton | our evdev doesn't support it yet, that's known | 07:34 |
| wgrant | Ah. | 07:36 |
| wgrant | And I guess that somebody will need to teach Synaptics about it. | 07:37 |
| wgrant | I can see a 'Device Enabled' property, and that's all. | 07:37 |
| tjaalton | it should support it as well | 07:37 |
| wgrant | Which doesn't appear in any of the listings that I've seen on the web. | 07:37 |
| tjaalton | I'll try my experimental evdev driver to make sure the plumbing is ok | 07:38 |
| tjaalton | yeah, evdev works so the synaptics driver is botched somehow | 07:41 |
| tjaalton | gone -> | 07:41 |
| wgrant | See you. | 07:41 |
| wgrant | Thanks. | 07:42 |
| tjaalton | wgrant: you could try patching out "#if GET_ABI_MAJOR(ABI_XINPUT_VERSION) >= 3", then it should work | 10:53 |
| tjaalton | from src/properties.c | 10:54 |
| tjaalton | wgrant: don't bother, I'll upload a new version | 11:26 |
| wgrant | tjaalton: Boom. | 14:31 |
| wgrant | tjaalton: dlopen: /usr/lib/xorg/modules/input//synaptics_drv.so: undefined symbol: SetProperty | 14:31 |
| tjaalton | great | 14:42 |
| wgrant | Indeed. | 14:44 |
| wgrant | So it does really need the new XI? | 14:44 |
| wgrant | Or is just getting confused? | 14:45 |
| tjaalton | I think it's the latter | 14:51 |
| tjaalton | anway, I'll revert the patch for now | 14:52 |
| tjaalton | that's what you get without hardware to test on | 14:57 |
| tjaalton | although, in this case loading the driver would've been enough | 14:57 |
| wgrant | I'll poke at it a bit and try to get it working. | 15:00 |
| tjaalton | for evdev to make it work with XI 1.5 the only thing it needed was to rip off the #ifdef's | 15:03 |
| wgrant | Brb, restarting with a hopefully-fixed driver. | 15:04 |
| tjaalton | and Bool SetProperty is defined in synaptics.c, that's why I think it's a problem with the driver code | 15:04 |
| wgrant | It's defined in properties.c, actually. | 15:07 |
| wgrant | Which probably wasn't being linked in. | 15:07 |
| Q-FUNK | ok, got around upgrading my development Geode workstation to Intrepid and I confirm: odd artifacts where the cursor is, using -geode on X 1.5 | 15:07 |
| wgrant | tjaalton: Right, got it. | 15:19 |
| wgrant | properties.o wasn't being linked in. | 15:19 |
| wgrant | Which probably means that src/Makefile.in wasn't being regenerated. | 15:20 |
| tjaalton | wgrant: yeah I saw the same.. | 15:27 |
| tjaalton | I'll fix the package, again, shortly | 15:28 |
| wgrant | tjaalton: Running automake fixes it, and produces just a tiny diff. | 15:28 |
| wgrant | And it works excellently. | 15:28 |
| wgrant | Thanks. | 15:28 |
| tjaalton | wgrant: excellent :) | 15:29 |
| tjaalton | dinner-> | 15:29 |
| wgrant | tjaalton: You wouldn't be able to upload the fix for bug #262292 as well, would you? | 15:34 |
| ubottu | Launchpad bug 262292 in xfree86-driver-synaptics "[intrepid] clicking is not working after merging" [High,Triaged] https://launchpad.net/bugs/262292 | 15:34 |
| tjaalton | wgrant: yep | 15:38 |
| wgrant | tjaalton: It is fixable through xinput now, but it's still annoying. | 15:40 |
| tjaalton | yes, it's been reverted upstream too | 15:40 |
| wgrant | And it for some reason won't let me set the necessary property to have the right number of items. | 15:40 |
| wgrant | What odd behaviour. | 15:43 |
| wgrant | I'm trying to override the tap actions through xinput, but it will only accept 6 arguments for that property, when it needs (and has initially) 7. If I give it a different name, it will accept 7 fine. | 15:44 |
| wgrant | Is that likely to be a driver or server bug? | 15:45 |
| tjaalton | I'm not sure, there are some limitations to the settings that can be changed atm | 15:45 |
| wgrant | I can change it fine, just not to the right length. | 15:47 |
| wgrant | No error is given, but it doesn't take effect, and watch-props doesn't see it. | 15:47 |
| tjaalton | maybe a driver bug | 15:48 |
| wgrant | Ah. | 15:48 |
| wgrant | It seems that anything that wants more than one will accept only one less than it should actually take. | 15:49 |
| * wgrant pokes in the code. | 15:49 | |
| tjaalton | jcristau: does autoreconf generate config.h.in? | 15:55 |
| jcristau | yeah, autoheader does, iirc | 15:56 |
| tjaalton | I'll autoreconfify synaptics, and noticed that the file is changed | 15:56 |
| * wgrant deletes lots of = | 15:57 | |
| wgrant | tjaalton: I've located and am fixing the off-by-one property issue, so xinput will be useful on it shortly. | 15:58 |
| wgrant | Will you accept such a patch? | 15:58 |
| tjaalton | wgrant: excellent | 15:58 |
| tjaalton | sure, if it works properly after that. You should also send it to xorg@ :) | 15:58 |
| wgrant | Of course, I shall. | 15:59 |
| wgrant | Testing now. | 15:59 |
| wgrant | Neeeed mooore baaandwidth. | 16:03 |
| tjaalton | I've got maybe 5kt/s | 16:04 |
| wgrant | I've 64Kbps for another day or two. | 16:05 |
| wgrant | kt? | 16:05 |
| tjaalton | oops | 16:05 |
| tjaalton | KB/s | 16:05 |
| wgrant | Ow. | 16:05 |
| tjaalton | GPRS ftw | 16:06 |
| wgrant | Sounds expensive. | 16:06 |
| wgrant | But I guess you're not in .au. | 16:07 |
| tjaalton | nah, maybe 10EUR /month | 16:07 |
| tjaalton | 3G (~128Kbps) where there is coverage | 16:07 |
| tjaalton | built-in adapter in the laptop :P | 16:08 |
| wgrant | 1.5c/KB on some of the more common plans here. | 16:08 |
| wgrant | For both 3G and GPRS. | 16:08 |
| tjaalton | uh | 16:08 |
| wgrant | It would go down a bit if you were to get a few hundred megabytes. But it's still very expensive. | 16:09 |
| wgrant | OK, now I've tested this trivial patch, and it works well. | 16:10 |
| tjaalton | cool, post it somewhere | 16:10 |
| wgrant | You need to delete a total of two characters. | 16:10 |
| wgrant | Line 458, line 470. >= should be just >. | 16:10 |
| wgrant | (src/properties.c) | 16:11 |
| tjaalton | if (prop->size >= MAX_CLICK and MAX_TAP? | 16:13 |
| wgrant | Yep. | 16:13 |
| tjaalton | ok, the driver built fine, uploading | 16:22 |
| tjaalton | three uploads in one day | 16:22 |
| wgrant | Thanks! | 16:23 |
| tjaalton | which is nice, Top10 here I come :) | 16:23 |
| wgrant | Heh. | 16:23 |
| tjaalton | so, send the patch to xorg@lists.freedesktop.org | 16:24 |
| wgrant | Any particular format/subject? | 16:26 |
| wgrant | I guess I'll check the archives. | 16:26 |
| tjaalton | I don't think it matters much | 16:26 |
| jcristau | 'git format-patch origin/master; git send-email' works | 16:31 |
| jcristau | i think xorg@ is subscriber-only though | 16:31 |
| * wgrant is rejected, so subscribes. | 16:36 | |
| * wgrant -> bed | 16:49 | |
| wgrant | Thanks for the uploads today, tjaalton. | 16:49 |
| tjaalton | wgrant: thanks for testing, and patches :) | 16:49 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!