Bug 229996

Summary: greek chars not displayed on startup
Product: [Fedora] Fedora Reporter: Stephanos Manos <stefmanos>
Component: initscriptsAssignee: Bill Nottingham <notting>
Status: CLOSED RAWHIDE QA Contact: Brock Organ <borgan>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: dimitris, rvokal, simos.bugzilla
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 8.52-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-04-13 22:30:38 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: 150225    
Attachments:
Description Flags
patch for this issue none

Description Stephanos Manos 2007-02-25 17:12:46 UTC 
Description of problem:
if system lang is Greek when booting in text mode or shutting down the localized
text is displayed as boxes.

Version-Release number of selected component (if applicable):
initscripts-8.51-1

How reproducible:
always

Steps to Reproduce:
1.change system lang to greek
2.shutdown, restart in text mode
  
Actual results:
Localized text displayed as box

Expected results:
The text to be readable

Additional info:
The problem is the additional lines in lang.sh to fix BZ# 229102.
Removing these lines (the If $LANG ...) then works as expected.
Comment 1 Stephanos Manos 2007-03-29 19:13:30 UTC 
ping ...
Comment 2 Simos Xenitellis 2007-03-30 01:01:37 UTC 
I checked bug 229102 and did not notice a patch/etc. I could see a reference
with not sourcing /etc/sysconfig/i18n though it is not clear whether this is
system-wide or how it can be made to affect only SSH session.

Could you please provide a patch or show what needs to be changed so this bug 
can be fixed?
Comment 3 Stephanos Manos 2007-03-30 09:18:45 UTC 
bug 229102 does not have a patch attached but as a result to fix it a If $LANG
... statement was added int he beginning of the the files /etc/profile.d/lang.sh
and /etc/profile.d/lang.csh

here is the beginning of the land.sh in f7
sourced=0

if [ -n "$LANG" ]; then
    sourced=1
else
    for langfile in /etc/sysconfig/i18n $HOME/.i18n ; do
    	[ -f $langfile ] && . $langfile && sourced=1
    done    
fi

and here is the line from fc6
sourced=0
for langfile in /etc/sysconfig/i18n $HOME/.i18n ; do
    [ -f $langfile ] && . $langfile && sourced=1
done  

if you remove the added if or more correctly(?) change it to 
if [ -n "$LANG" ]; then
    sourced=0
else
...

then the problem is solved
Comment 4 Dimitris Glezos 2007-04-04 13:53:47 UTC 
Bill, any update on this?
Comment 5 Simos Xenitellis 2007-04-05 05:09:30 UTC 
Stephanos, prepei na steileis patches (diff -u) gia ta dio arxeia, diaforetika
de 8a ginei tipota.
Comment 6 Bill Nottingham 2007-04-13 22:30:01 UTC 
Created attachment 152595 [details]
patch for this issue

Try this - basically, we do need to re-read only when we're calling
unicode_start to set the font.
Comment 7 Bill Nottingham 2007-04-13 22:30:38 UTC 
Will be in 8.52-1.
Comment 8 Dimitris Glezos 2007-04-14 00:08:51 UTC 
Does this issue have anything in common with bug #222549?

Although this might not be the best place to ask this: do we have wiki pages
with instructions on how to convert such a patch to an SRPM and test it and info
about where these sources lie? (found out the CVS root of initscripts through
http://rhlinux.redhat.com/initscripts/, which I didn't know it existed until today.)
Comment 9 Bill Nottingham 2007-04-14 00:19:11 UTC 
In as much as you also need to have SYSFONT set to something appropriate for
Greek, it's related. You can always set the font by hand, though.

As for the patch, you can add it as PatchN: just like in any other RPM.
Comment 10 Stephanos Manos 2007-04-15 16:10:15 UTC 
(In reply to comment #9)
> In as much as you also need to have SYSFONT set to something appropriate for
> Greek, it's related. You can always set the font by hand, though.
Related only if you change language after installation.


As for the patch It works