[vim/vim] git regression: keywordprg is not set to git show anymore (Issue #9845)
Jiri Slaby
Steps to reproduce
- Open vim in a git directory
- Paste a SHA (or use
git rebase -i @^^) - Press shift-k.
Expected behaviour
git show is run with the above SHA. But man is run instead. This is apparently a result of 2f0936c#diff-e92ab7d5e3bfe3e88cfb645f5ce7d4e05cc4f6495d72e8458e97f95c229fdb7e
Why was the file removed?
Version of Vim
8.2.4375
Environment
- openSUSE Tumbleweed
- xterm
- xterm-256color
- bash
Logs and stack traces
No response
—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Jiri Slaby
This is coming from tpope/vim-git@6ee5e1f . @tpope, why do you think there is no good idea there? Is there any replacement now?
—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Christian Brabandt
this comes from #9477 I'd suggest to discuss this with @tpope at his upstream repo: https://github.com/tpope/vim-git
—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Tim Pope
This was never an officially documented feature, just an ill-conceived idea that leaked out of ftplugin/git.vim. If you want to bring it back, you can add it yourself in after/ftplugin/gitrebase.vim:
setlocal keywordprg=git\ show
—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
David Disseldorp
This was never an officially documented feature, just an ill-conceived idea that leaked out of
ftplugin/git.vim. If you want to bring it back, you can add it yourself inafter/ftplugin/gitrebase.vim:setlocal keywordprg=git\ show
Thanks for the workaround. I'm also a heavy user of this functionality (it makes git rebase -i really nice) so had hoped to see the regression fixed, but I guess I'll have to live with the local modification.
—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Christian Brabandt
closing then.
—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Christian Brabandt
Closed #9845.
—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Jiri Slaby
@ddiss: https://build.opensuse.org/request/show/960651
—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.