Bug 484217

Summary: [PATCH] only call fedora-setup-keyboard for keyboards
Product: [Fedora] Fedora Reporter: Adel Gadllah <adel.gadllah>
Component: xorg-x11-serverAssignee: X/OpenGL Maintenance List <xgl-maint>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: 10CC: harald, liedekef, peter.hutterer, xgl-maint
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 1.5.3-13.fc10 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-02-05 17:41:45 EST Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Attachments:
Description Flags
Patch against the fdi file we ship in X none

Description Adel Gadllah 2009-02-05 09:18:47 EST
Created attachment 330987 [details]
Patch against the fdi file we ship in X

Currently we run fedora-setup-keyboard for every device that has keys (capability input.keys[1]) which results into useless calls for devices that has only one button (Power Button, Sleep Button, etc)

Changing the keyboard layout for such devices is useless, because they don't respect any layout anyway.

The attached patch fixes this by checking for the capability input.keyboard.


1; Well we are checking if input.xkb.layout exists but hal adds this for every device that has the capability input.keys in /usr/share/hal/fdi/policy/10osvendor/10-keymap.fdi 

With this patch everything should work as expected with the small benefit that we only call fedora-setup-keyboard once for most system (unless multiple keyboards are detected) which should speed up the boot process.

See https://bugzilla.redhat.com/show_bug.cgi?id=476028
Comment 1 Peter Hutterer 2009-02-05 17:41:45 EST
Applied in xorg-x11-server-1.5.99.902-3. Thanks for the patch.
Comment 2 Fedora Update System 2009-02-24 15:45:57 EST
xorg-x11-server-1.5.3-13.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 3 Franky Van Liedekerke 2009-02-26 16:39:21 EST
Apparently, this fix causes more troubles than it solves, see:

https://bugzilla.redhat.com/show_bug.cgi?id=484488
Comment 4 Peter Hutterer 2009-03-13 00:41:06 EDT
Fix in F-10 has been reverted because of bug 484488 which we couldn't narrow down.
The same fix is in rawhide though where the conflicts between hal, x, gdm and gnome are resolved differently and 484488 hopefully does not have any effect.