Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 212844 - sys-apps/man-1.6f-r1 breaks vimmanpager
Summary: sys-apps/man-1.6f-r1 breaks vimmanpager
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Vim Maintainers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-09 17:27 UTC by Guillaume Castagnino
Modified: 2008-04-06 09:23 UTC (History)
0 users

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


Attachments
vimmanpager-fix.patch (vimmanpager-fix.patch,393 bytes, patch)
2008-03-09 17:28 UTC, Guillaume Castagnino
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Guillaume Castagnino 2008-03-09 17:27:20 UTC
Since sys-apps/man-1.6f-r1, NROFF is now "/usr/bin/nroff -mandoc"

Changelog :
*man-1.6f-r1 (24 Feb 2008)
  +files/man-1.6f-unicode.patch, +man-1.6f-r1.ebuild:
  Do not force -c/-T anymore to groff so that it can work its locale magic
  #146315 by Matthias Schwarzott.


Impact removing the '-c' option is that color control codes are pushed in stdout.
This is not a problem with the standard manpager, but it corrupts the vimmanpager display : vim displays the raw codes in the man pages, since control color codes are not interpreted.


I see 2 solutions to fix this :
1) reintroduce the '-c' option to NROFF (in /etc/man.conf)
2) fix the vimmanpager script (patching the vim.eclass), by stripping the color control codes in the stdin stream. Follow a patch that fix this. I think that this should be the best solution.
Comment 1 Guillaume Castagnino 2008-03-09 17:28:40 UTC
Created attachment 145662 [details, diff]
vimmanpager-fix.patch

This patch apply on vim.eclass that generate the vimmanpager script.

There should be other way to do the same thing (stripping the color control codes), but this way works well.
Comment 2 Ali Polatel (RETIRED) gentoo-dev 2008-04-06 09:23:39 UTC
The fix is in CVS. Thanks for reporting :)