MikeM wrote:
It's been a while since I reported a problem with the patch here and no
further comment has been made. I guess the majority of users out there
are probably using one of the pre-packaged versions of vim 7.3 and so
this problem hasn't come down the pipeline to hit them yet. I have seen
one other reference to the problem whilst browsing the web:
http://crumbtrail.chesmart.in/

I still don't think the original patch was the right solution here, but
until someone comes up with a better solution, I wonder if the the
following patch could be applied?

--- a/ftplugin/man.vim2011-07-22 19:55:14.000000000 +0100
+++ b/ftplugin/man.vim2011-07-24 17:55:49.000000000 +0100
@@ -17,7 +17,9 @@
" Ensure Vim is not recursively invoked (man-db does this)
" when doing ctrl-[ on a man page reference.
- let $MANPAGER = ""
+ if exists("$MANPAGER")
+ let $MANPAGER = ""
+ endif
" allow dot and dash in manual page name.
setlocal iskeyword+=\.,-

This patch maintains the fix for the original problem, but limits the
potential side effects to those that have a MANPAGER environment
variable actually defined.

I have no objection to this patch. so, Ack.

BTW, I'm not sure man.vim should include such lines at all. Original reporter already found a workaround for his problem. And his patch caused side-effect for others.

Regards,
namsh

--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

Raspunde prin e-mail lui