Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 127207

Summary: [PATCH] keymap "fi" does not work properly
Product: [Fedora] Fedora Reporter: Jarkko <jval>
Component: rdesktopAssignee: Mark McLoughlin <markmc>
Status: CLOSED RAWHIDE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 2CC: pmatilai, scop, wtogami
Target Milestone: ---Keywords: i18n, Patch
Target Release: ---   
Hardware: i686   
OS: Linux   
Whiteboard:
Fixed In Version: 1.3.1-5 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2004-09-24 15:50:23 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 123268, 125270    

Description Jarkko 2004-07-04 00:39:02 UTC
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.6) Gecko/20040510

Description of problem:
The "fi" keymap does not work properly. For example I can't get "&" at
all.

Version-Release number of selected component (if applicable):
rdesktop-1.3.1-3

How reproducible:
Always

Steps to Reproduce:
1. choose the "fi" keymap
2. connect to some Windows server
3. try to push "&"
    

Actual Results:  Nothing happens.

Expected Results:  The key should print "&" on the screen as normally.

Additional info:

Comment 1 Warren Togami 2004-07-04 05:06:41 UTC
Can anybody else confirm this problem?

Comment 2 Ville Skyttä 2004-07-04 09:20:22 UTC
Confirmed.  The keys @&<>' do not work in the shipped fi keymap for
me.  I have already posted a patch that fixes this for me upstream, it
would be very nice to get this applied in RH/FC.
http://sourceforge.net/tracker/index.php?func=detail&aid=959504&group_id=24366&atid=381349

Comment 3 Jarkko 2004-07-05 05:27:59 UTC
Applied the patch. Everyhing seems to be working ok now, but I'm
getting the following warning:

WARNING: No translation for (keysym 0xfe03, ISO_Level3_Shift)

Comment 4 Jarkko 2004-07-05 05:36:46 UTC
Hmm, that warning must have been there already because the patch
contained only additions. 

It might be a good idea though to check out what key should that
(keysym 0xfe03, ISO_Level3_Shift) be so that we get rid of that warning.

Comment 5 Ville Skyttä 2004-07-05 06:42:44 UTC
Actually, the patch removes 5 mappings (ie. those of the keys that did
not work, some of which are duplicated in the original), it does not
add anything.

Comment 6 Jarkko 2004-07-05 06:52:01 UTC
Oh, I was diffing the files wrong way. :)

So I'm not sure if that warning was there before this patch or not. 
Propably it was (if I understand the keymap files correctly).

Comment 7 Ville Skyttä 2004-07-05 16:41:40 UTC
My patch does not affect the ISO_Level3_Shift warning, it was there
already before.  AFAICS it's purely cosmetic, harmless warning, but
nevertheless fixed in upstream CVS:
http://cvs.sourceforge.net/viewcvs.py/rdesktop/rdesktop/keymaps/modifiers?r1=1.5&r2=1.6&diff_format=u

It would be nice to include that fix as well in FC.


Comment 8 Ville Skyttä 2004-07-05 16:42:43 UTC
(Oh, and ISO_Level3_Shift is AltGr, without the upstream patch a
warning is printed every time it is pressed.)

Comment 9 Jarkko 2004-07-06 06:36:05 UTC
Thanks for that fix. Everything is now working exactly like it should.

So just release a new rpm in updates with these two patches applied
and this bug can be closed. :)

By the way, that second ISO_Level3_Shift patch link says the following
(I had to download the whole file instead): 

===================================================================
RCS file: /cvsroot/rdesktop/rdesktop/keymaps/modifiers,v
retrieving revision 1.5
Unable to create temp directory: Permission denied

Comment 10 Ville Skyttä 2004-07-08 17:37:41 UTC
Yeah, sf.net has the habit of breaking their CVS and/or CVSweb far too
often.  Anyway, the diff link in comment 7 works now; could we get
these fixes in, at least for FC3, even better if also a FC2 erratum? 
That would result in many happy Finns :)

Comment 11 Mark McLoughlin 2004-09-24 15:50:23 UTC
Okay, warren seems to have added this patch. Seems to be in
rdesktop-1.3.1-5 which is in FC3 test2.