[03:43] <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:45] <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?)
[07:31] <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:34] <tjaalton> our evdev doesn't support it yet, that's known
[07:36] <wgrant> Ah.
[07:37] <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:38] <tjaalton> I'll try my experimental evdev driver to make sure the plumbing is ok
[07:41] <tjaalton> yeah, evdev works so the synaptics driver is botched somehow
[07:41] <tjaalton> gone ->
[07:41] <wgrant> See you.
[07:42] <wgrant> Thanks.
[10:53] <tjaalton> wgrant: you could try patching out "#if GET_ABI_MAJOR(ABI_XINPUT_VERSION) >= 3", then it should work
[10:54] <tjaalton> from src/properties.c
[11:26] <tjaalton> wgrant: don't bother, I'll upload a new version
[14:31] <wgrant> tjaalton: Boom.
[14:31] <wgrant> tjaalton: dlopen: /usr/lib/xorg/modules/input//synaptics_drv.so: undefined symbol: SetProperty
[14:42] <tjaalton> great
[14:44] <wgrant> Indeed.
[14:44] <wgrant> So it does really need the new XI?
[14:45] <wgrant> Or is just getting confused?
[14:51] <tjaalton> I think it's the latter
[14:52] <tjaalton> anway, I'll revert the patch for now
[14:57] <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
[15:00] <wgrant> I'll poke at it a bit and try to get it working.
[15:03] <tjaalton> for evdev to make it work with XI 1.5 the only thing it needed was to rip off the #ifdef's
[15:04] <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:07] <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:19] <wgrant> tjaalton: Right, got it.
[15:19] <wgrant> properties.o wasn't being linked in.
[15:20] <wgrant> Which probably means that src/Makefile.in wasn't being regenerated.
[15:27] <tjaalton> wgrant: yeah I saw the same..
[15:28] <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:29] <tjaalton> wgrant: excellent :)
[15:29] <tjaalton> dinner->
[15:34] <wgrant> tjaalton: You wouldn't be able to upload the fix for bug #262292 as well, would you?
[15:38] <tjaalton> wgrant: yep
[15:40] <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:43] <wgrant> What odd behaviour.
[15:44] <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:45] <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:47] <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:48] <tjaalton> maybe a driver bug
[15:48] <wgrant> Ah.
[15:49] <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:55] <tjaalton> jcristau: does autoreconf generate config.h.in?
[15:56] <jcristau> yeah, autoheader does, iirc
[15:56] <tjaalton> I'll autoreconfify synaptics, and noticed that the file is changed
[15:57]  * wgrant deletes lots of =
[15:58] <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:59] <wgrant> Of course, I shall.
[15:59] <wgrant> Testing now.
[16:03] <wgrant> Neeeed mooore baaandwidth.
[16:04] <tjaalton> I've got maybe 5kt/s
[16:05] <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:06] <tjaalton> GPRS ftw
[16:06] <wgrant> Sounds expensive.
[16:07] <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:08] <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:09] <wgrant> It would go down a bit if you were to get a few hundred megabytes. But it's still very expensive.
[16:10] <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:11] <wgrant> (src/properties.c)
[16:13] <tjaalton> if (prop->size >= MAX_CLICK and MAX_TAP?
[16:13] <wgrant> Yep.
[16:22] <tjaalton> ok, the driver built fine, uploading
[16:22] <tjaalton> three uploads in one day
[16:23] <wgrant> Thanks!
[16:23] <tjaalton> which is nice, Top10 here I come :)
[16:23] <wgrant> Heh.
[16:24] <tjaalton> so, send the patch to xorg@lists.freedesktop.org
[16:26] <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:31] <jcristau> 'git format-patch origin/master; git send-email' works
[16:31] <jcristau> i think xorg@ is subscriber-only though
[16:36]  * wgrant is rejected, so subscribes.
[16:49]  * wgrant -> bed
[16:49] <wgrant> Thanks for the uploads today, tjaalton.
[16:49] <tjaalton> wgrant: thanks for testing, and patches :)