Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 65203

Summary: less is broken for Japanese
Product: [Retired] Red Hat Linux Reporter: Nakai <ynakai>
Component: lessAssignee: Karsten Hopp <karsten>
Status: CLOSED RAWHIDE QA Contact: Aaron Brown <abrown>
Severity: medium Docs Contact:
Priority: medium    
Version: 7.3CC: eng-asia-list, wtogami
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2002-05-20 09:12:57 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:
Attachments:
Description Flags
'ABC' file in Japanese multibyte.
none
It should be like this on kterm
none
less-358+iso247-20001210-rh.diff (Updated iso patch for less) none

Description Nakai 2002-05-20 07:55:50 UTC 
From Bugzilla Helper:
User-Agent: Mozilla/5.0 Galeon/1.2.0 (X11; Linux i686; U;) Gecko/20020408

Description of problem:
less is broken for Japanese.
The attached file should be seen as the attached png in
Japanese mode. (LANG=ja_JP.eucJP, JLESSCHARSET=japanese)

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


How reproducible:
Always

Steps to Reproduce:
1. less the attached file.
2. Oops.
3. Do you say we need to use emacs instead of less? RHL7.2 was better..
	

Actual Results:  Broken - it shows hex dump.

Expected Results:  Human readable.

Additional info:
Comment 1 Nakai 2002-05-20 07:56:50 UTC 
Created attachment 57964 [details]
'ABC' file in Japanese multibyte.
Comment 2 Nakai 2002-05-20 08:00:05 UTC 
Created attachment 57965 [details]
It should be like this on kterm
Comment 3 Nakai 2002-05-20 09:01:26 UTC 
I understand this iso patch breaks -N option for Japanese and other
languages. And I found a mistake in the patch. The patch author
added a line but forget to add {} to the for sentence. Below change
fixed the problem.

--- less-358/line.c.orig        Mon May 20 18:02:51 2002
+++ less-358/line.c     Mon May 20 18:03:06 2002
@@ -152,9 +152,10 @@
        {
                sprintf(&linebuf[curr], "%*d", LINENUM_WIDTH, lno);
                column += LINENUM_WIDTH;
-               for (i = 0;  i < LINENUM_WIDTH;  i++)
+               for (i = 0;  i < LINENUM_WIDTH;  i++) {
                        charset[curr] = ASCII;
                        attr[curr++] = AT_NORMAL;
+               }
        }
        /*
         * Append enough spaces to bring us to the lmargin.
Comment 4 Nakai 2002-05-20 09:12:14 UTC 
Created attachment 57984 [details]
less-358+iso247-20001210-rh.diff (Updated iso patch for less)
Comment 5 Nakai 2002-05-20 09:12:51 UTC 
Updated iso patch attached.
Comment 6 Karsten Hopp 2002-06-18 23:26:49 UTC 
fixed in less-358-27 with iso254  and other patches