Attachment 107962 Details for Bug 85114 – 0002-macintosh_vndr-us-include-keypad-overlay-explicitly.patch

[patch] 0002-macintosh_vndr-us-include-keypad-overlay-explicitly.patch

0002-macintosh_vndr-us-include-keypad-overlay-explicitly.patch (text/plain), 1.20 KB, created by Ran Benita on 2014-10-16 22:30:49 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Ran Benita

Created: 2014-10-16 22:30:49 UTC

Size: 1.20 KB

patch

obsolete

>From 652de1cb841082dd8c949412545be2931da135a9 Mon Sep 17 00:00:00 2001
>From: Ran Benita <ran234@gmail.com>
>Date: Thu, 16 Oct 2014 19:52:08 +0300
>Subject: [PATCH 2/3] macintosh_vndr/us: include "keypad(overlay)" explicitly
>
>With current xkbcomp, include statements do not obey default annotations
>(it does obey them when given a file directly), but libxkbcommon does.
>So in xkbcomp, `include "keypad"` includes the first map in the keypad
>file, which is "overlay", but includes the default in libxkbcommon,
>which is "x11".
>
>"overlay" might in fact be wrong, especially since the keycodes
>necessary for the overlay to work are not defined anywhere. But
>some users may rely on this in some way, so let's not change it.
>
>Signed-off-by: Ran Benita <ran234@gmail.com>
>---
> symbols/macintosh_vndr/us | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/symbols/macintosh_vndr/us b/symbols/macintosh_vndr/us
>index d4e4af1..58bfee5 100644
>--- a/symbols/macintosh_vndr/us
>+++ b/symbols/macintosh_vndr/us
>@@ -59,7 +59,7 @@ default xkb_symbols "extended" {
>     key <SPCE> {	[  space			]	};
> 
>     include "eurosign(e)"
>-    include "keypad"
>+    include "keypad(overlay)"
> 
>     key <KPEQ> {	[  equal			]	};
> };
>-- 
>2.1.2
>

Actions: View | Splinter Review

Attachments on bug 85114: 107961 | 107962 | 107963

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.