<div dir="ltr"><div class="gmail_extra">Hi,</div><div class="gmail_extra"><br><div class="gmail_quote">On 18 August 2014 13:14, Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Mon, 18 Aug 2014 14:57:37 +0300<br>
Giulio Camuffo <<a href="mailto:giuliocamuffo@gmail.com">giuliocamuffo@gmail.com</a>> wrote:<br>
> 2014-08-18 14:35 GMT+03:00 Pekka Paalanen <<a href="mailto:ppaalanen@gmail.com">ppaalanen@gmail.com</a>>:<br>
> > had these patches which I'd like to see going in:<br>
> <a href="http://lists.freedesktop.org/archives/wayland-devel/2013-December/012589.html" target="_blank">http://lists.freedesktop.org/archives/wayland-devel/2013-December/012589.html</a><br>
<br>
</div>This one looks pretty simple, and I can confirm the problem with<br>
CapsLock, so yes, I'd like to take this. Could you rebase it?<br></blockquote><div><br></div><div>This is a good start, yes. It's either that or go through the keymap state trying to work out the modifier state to accompany the LEDs, which is doable but a bit fiddly.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> <a href="http://lists.freedesktop.org/archives/wayland-devel/2013-December/012590.html" target="_blank">http://lists.freedesktop.org/archives/wayland-devel/2013-December/012590.html</a><br>
> <a href="http://lists.freedesktop.org/archives/wayland-devel/2013-December/012591.html" target="_blank">http://lists.freedesktop.org/archives/wayland-devel/2013-December/012591.html</a><br>
<br>
Of these two, the first needs the second to be used at all, right? The<br>
follow-up comment is true, I would not want NumLock be on by default on<br>
my laptop, so I think this needs more work, like you have identified.<br></blockquote><div><br></div><div>Yes, definitely not on by default please. As for the second patch, I'm a little hesitant to add it to X11 in particular: why should we be telling the host server what the modifier state for the entire session (which is larger than the nested compositor) should be?</div>
<div><br></div><div>evdev requires LEDs to be driven because in that situation, we are the lowest-level input consumer, and thus calculate our own state (i.e. call xkb_state_update_key). In nested environments (compositor-{x11,wayland}), we never calculate our own state, but receive the state from the host display server. If the host display server isn't keeping the state sent to its clients in sync with the state of its input devices, then that's a bug.</div>
<div><br></div><div>(tl;dr: NAK)</div><div><br></div><div>Cheers,</div><div>Daniel <br></div></div></div></div>