Bug 484878 - Missing keys in tty_named_keys.h
Summary: Missing keys in tty_named_keys.h
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: audit
Version: 10
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Steve Grubb
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-02-10 13:38 UTC by Miloslav Trmač
Modified: 2009-07-14 14:47 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-07-14 14:47:40 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Add keys for terminals from ncurses-base (12.41 KB, patch)
2009-03-18 19:56 UTC, Miloslav Trmač
no flags Details | Diff

Description Miloslav Trmač 2009-02-10 13:38:54 UTC
There are probably many missing keys, at least the following should be added:

esc O A    up
esc O B    down
esc O C    right
esc O D    left

(seen in #483086)

Comment 1 Steve Grubb 2009-02-10 21:47:02 UTC
I updated the mappings for the above 4 codes in svn commit 249. Are there any more that should be done? 

I noticed that the original mappings work fine for ssh sessions and the ones in the description above seem to come from a gdm session. Would the terminal emulators employed by each cause the mappings to be different? If so, I wonder how we solve this problem reliably?

Comment 2 Miloslav Trmač 2009-02-10 21:56:20 UTC
Best would be to use terminfo, but $TERM is not available in the message.  Perhaps we target just the terminals in /lib/terminfo (chosen as important by the ncurses maintainer) and add all input sequences for these.

Comment 3 Steve Grubb 2009-02-10 22:11:36 UTC
OK, I will wait for further suggestions on this one.

Comment 4 Miloslav Trmač 2009-03-18 19:56:53 UTC
Created attachment 335756 [details]
Add keys for terminals from ncurses-base

The attached patch adds all keys described in terminfo in the ncurses-base package; there are some unavoidable conflicts, in which case the Linux variant, or the more popular variant (measured by the number of different terminal descriptions, not by actual usage) was chosen.

The patch also fixes an incorrect leading comma characters produced by aureport.

Comment 5 Miloslav Trmač 2009-03-18 19:58:22 UTC
(I didn't do any performance testing, the linear search might eventually be found too slow.  In that case the keys should probably be built in to a trie.)

Comment 6 Steve Grubb 2009-04-06 14:31:02 UTC
Applied to svn.

Comment 7 Steve Grubb 2009-07-14 14:47:40 UTC
closing since this was pushed out.


Note You need to log in before you can comment on or make changes to this bug.