bugGNU GRUB - Bugs: bug #38242, grub-mklayout won't handle the...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #38242: grub-mklayout won't handle the keys to input a backslash character on Japanese keyboard layout

Submitter:  Morita Sho <moritasho>
Submitted:  Mon 04 Feb 2013 01:49:36 AM UTC
   
 
Category:  Terminal Severity:  Major
Priority:  5 - Normal Item Group:  Software Error
Status:  Fixed Privacy:  Public
Assigned to:  None Originator Name: 
Open/Closed:  Closed Release:  Bazaar - trunk
Release:  Reproducibility:  Every Time
Planned Release:  None

Mon 04 Feb 2013 01:49:36 AM UTC, original submission:  

Hi,

I noticed that grub-mklayout rejects the keys to input a backslash on Japanese keyboard layout.
Executing grub-mklayout for Japanese keymap will outputs the following message:


  % ckbcomp jp | grub-mklayout -o /dev/null
  Unknown keycode 0x54
  Unknown keycode 0x65
  Unknown keycode 0x7f
  Unknown keycode 0x7c   <==
  Unknown keycode 0x79
  Unknown keycode 0x59   <==


The keys 0x7c(124) and 0x59(89) are necessary to input a backslash character on Japanese keyboard layout.

The output of ckbcomp for these keys are as follow:


  % ckbcomp jp | grep '124\|89'
  keycode 124 = U+005c U+007c U+005c U+007c ...
  keycode 89 = U+005c U+005f U+005c U+005f ...



Here is the information about these keys:

Linux keycode: 0x59(89), defined as KEY_RO in include/uapi/linux/input.h
AT Set 1 Scancode: 0x73
AT Set 2 Scancode: 0x51
USB Usage ID: 0x87, Keyboard International1

Linux keycode: 0x7C(124), defined as KEY_YEN in include/uapi/linux/input.h
AT Set 1 Scancode: 0x7D
AT Set 2 Scancode: 0x6A
USB Usage ID: 0x89, Keyboard International3

References:



I think some modifications to util/grub-mklayout.c, grub-core/term/at_keyboard.c and include/grub/keyboard_layouts.h is needed to add support for those keys.  I made and attached a patch.  But I'm afraid for that this patch might breaks backward compatibility as it changes the value of GRUB_KEYBOARD_LAYOUTS_ARRAY_SIZE...


Regards,

Morita Sho <moritasho>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #27377:  patch added by moritasho (4KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by phcoder (Updated the item)
  • -email is unavailable- added by moritasho (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-04-12 phcoder StatusNone Fixed
        Open/ClosedOpen Closed
    2013-02-04 moritasho Attached File- Added patch, #27377

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code