Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 150169 - app-editors/nvi has trouble loading unicode files
Summary: app-editors/nvi has trouble loading unicode files
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Harald van Dijk (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-05 07:39 UTC by Karl Hakimian
Modified: 2006-10-06 06:59 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
patch to fix unicode read in nvi (nvi-1.81.5-wide.patch,3.54 KB, patch)
2006-10-05 07:41 UTC, Karl Hakimian
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Karl Hakimian 2006-10-05 07:39:34 UTC
nvi does not use the correct call to read wide characters from a file. I have patched it and submitted the patch upstream and told the author that I would also submit the patch to the gentoo developers.
Comment 1 Karl Hakimian 2006-10-05 07:41:02 UTC
Created attachment 98842 [details, diff]
patch to fix unicode read in nvi

I have tried this patch against both the stable version and the soon to be released eselect version of nvi. It works for me.
Comment 2 Harald van Dijk (RETIRED) gentoo-dev 2006-10-06 04:50:36 UTC
While that patch seems sane to me, could you give a quick example of when it is necessary? (I can use nvi to edit my UTF-8 encoded files, without problems yet.)
Comment 3 Karl Hakimian 2006-10-06 05:02:55 UTC
Create a file containing utf-8 characters. Save it. Bring up nvi and then load that file in via :r file. The utf-8 will be garbled.
Comment 4 Harald van Dijk (RETIRED) gentoo-dev 2006-10-06 05:28:31 UTC
Thanks, fixed in 1.81.5-r5. I know that's in package.mask, but eselect-vi is supposed to be unmasked soon. If it's not, I'll add another non-eselect revision as well.
Comment 5 Karl Hakimian 2006-10-06 06:59:10 UTC
Perfect. Thanks.