The current nvi port still contains a lot of bugs related to wide character support that is enabled by the unicode-useflag. (For example, using the '#' command crashes nvi even in all-ASCII files.) I noticed Sven Verdoolaege's Git repository (see: http://repo.or.cz/w/nvi.git) already contains fixes for a lot of these issues; I've collected the relevant changes into a big patch file that I'll attach. Would a developer please give this a quick look at this and then update the ebuild to apply this patch? Reproducible: Always Steps to Reproduce: nvi a123^[ ^#+ Actual Results: Segmentation fault Expected Results: 124
Created attachment 345646 [details, diff] Patch to fix several --enable-widechar bugs
(In reply to Maks Verver from comment #1) > Created attachment 345646 [details, diff] [details, diff] > Patch to fix several --enable-widechar bugs Hi Maks, Thanks for taking the time and doing this. However, this is a major patch and I wonder how much have you actually tested it. Do you use NVI regularly? for how long have you used it with this patch? I'd probably release a masked version of NVI with this patch an see if it doesn't break anything else. Thanks!
I've been using the patched version for half a year without problems, and although I do use vi quite a lot, I cannot guarantee my typical usage patterns provide adequate test coverage. Since the current version in portage has known problems, I would say it makes more sense to add the patch and deal with whatever problems arise than to keep a known-buggy version around. The changes in the patch seem sensible enough, so I wouldn't expect too many complications.
Felix, you're the maintainer of this package. Have you reviewed this patch?
I could not reproduce the report. Please provide the requested information.
felix, could you reproduce it?
Ping.