NetBSD-specific changes for xf86-input-keyboard
Alan Coopersmith
alan.coopersmith at oracle.com
Fri Jul 26 18:40:33 PDT 2013
On 07/26/13 02:23 PM, Thomas Klausner wrote:
> These patches mirror the commit history of changes done to
> xf86-input-keyboard in the NetBSD xsrc repository (except perhaps
> for timing between patches for different files).
>
> I'm willing to squash or edit patches to reduce the number of diffs;
> please advise on what you want combined in this way.
>
> Other suggestions welcome too, for example for patch 21's autoconf
> code for machine architecture detection.
For patches 1-7, 9-11, while I can't verify you've chosen the right
values for those keycodes, I assume you and your users have already tested
that out, and since the way you've set them looks fine:
Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>
Patches 8, 12, 18, seem to require more platform specific knowledge
than I have of your platform. They don't seem wrong, I just don't know if
they're right. Since they're credited to other authors, would you consider
yourself to have reviewed them and be willing to offer up the r-b for them?
Patches 13, 15, 21 I replied directly to with comments before giving r-b.
Patches 14, 16, 17, 19, 20 seem simple enough to review without platform
knowledge, and look fine to me, so those can also have:
Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>
Since this is a huge pile of patches, and you've been upstreaming a lot lately,
is it time to discuss git commit access so you can push them yourself instead
of having to wait both for someone to review and for a committer to push?
--
-Alan Coopersmith- alan.coopersmith at oracle.com
Oracle Solaris Engineering - http://blogs.oracle.com/alanc
More information about the xorg-devel
mailing list