Summary: | app-editors/nvi-1.81.6-r3: various wide character bugs with USE=unicode | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Maks Verver <maksverver> |
Component: | Current packages | Assignee: | Felix Janda <felix.janda> |
Status: | RESOLVED WONTFIX | ||
Severity: | normal | CC: | jstein, mgorny, proxy-maint, saint.snit, treecleaner |
Priority: | Normal | Keywords: | PATCH, PMASKED, PullRequest |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://github.com/gentoo/gentoo/pull/10831 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | 690102 | ||
Bug Blocks: | |||
Deadline: | 2020-03-18 | ||
Attachments: | Patch to fix several --enable-widechar bugs |
Description
Maks Verver
2013-04-15 22:03:30 UTC
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. |