New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
vimPlugins.notational-fzf-vim: patch plugin to fix missing runtime dependency #80657
Conversation
Did you run some kind of formatter? There are a lot of irrelevant changes in the diff. |
Yeah, sorry, ran nix-fmt, reverting. |
So, I'm not sure if this adds rg to the path, I do know that the plugin runs execute!("rg") rather than having it coded in, so adding ripgrep to PATH made the most sense to me. Idk if buildinputs does that, or if something else is necessary? |
No, Unfortunately we don't currently have an easy way to do this at runtime. It could be done in combination with plugin management (simply adding an appropriate snippet to the generated vimrc), but we don't currently do that. To make sure ripgrep can be found by the plugin, you will need to patch the plugin's source to contain an absolute path to ripgrep. You can do that |
python3 executable patched in too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please clean up git history, should be a single commit
ab9ec10
to
ebd9397
Compare
Okay, that should do it, tested and working. Thanks for patience with experimentation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Though keep in mind that this fix is pretty brittle, which is why appending to PATH is somewhat nicer. Still, this is a clear improvement and can be merged if @jonringer agrees.
Thanks! |
No description provided.