[bug#49359] [PATCH] services: xorg: Do not force driver for keyboard defaults.

[bug#49359] [PATCH] services: xorg: Do not force driver for keyboard defaults.

[Brice Waegeneire]

Let xorg choose the correct driver for keyboards; if need be the driver
can be forced by the user on a device by device basis.

* gnu/services/xorg.scm (xorg-configuration->file)[input-class-section]:
Remove matching on device path and forcing the driver.  Put each option
on a separate line.
---
 gnu/services/xorg.scm | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/gnu/services/xorg.scm b/gnu/services/xorg.scm
index 8ffea3b9dd..2c894ac6b9 100644
--- a/gnu/services/xorg.scm
+++ b/gnu/services/xorg.scm
@@ -225,27 +225,24 @@ EndSection"))
               (define (input-class-section layout variant model options)
                 (string-append "
 Section \"InputClass\"
-  Identifier \"evdev keyboard catchall\"
+  Identifier \"keyboard defaults\"
   MatchIsKeyboard \"on\"
-  Option \"XkbLayout\" " (object->string layout)
+  Option \"XkbLayout\" " (object->string layout) "\n"
   (if variant
       (string-append "  Option \"XkbVariant\" \""
-                     variant "\"")
+                     variant "\"\n")
       "")
   (if model
       (string-append "  Option \"XkbModel\" \""
-                     model "\"")
+                     model "\"\n")
       "")
   (match options
     (()
      "")
     (_
      (string-append "  Option \"XkbOptions\" \""
-                    (string-join options ",") "\""))) "
-
-  MatchDevicePath \"/dev/input/event*\"
-  Driver \"evdev\"
-EndSection\n"))
+                    (string-join options ",") "\"\n")))
+  "EndSection\n"))
 
               (define (expand modules)
                 ;; Append to MODULES the relevant /lib/xorg/modules
-- 
2.31.1

[bug#49359] [PATCH] services: xorg: Do not force driver for keyboard defaults.

[Tobias Geerinckx-Rice via Guix-patches via]

[-- Attachment #1: Type: text/plain, Size: 407 bytes --]

Brice,

Brice Waegeneire 写道：
> Let xorg choose the correct driver for keyboards; if need be the 
> driver
> can be forced by the user on a device by device basis.

Thanks, I think this is the right thing to do.

I hope someone will test this who (1) still uses ‘real’ Xorg (2) 
doesn't use just the HIDs that came free with their laptop :-)  I 
do neither.

LGTM otherwise!

T G-R

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]