Skip to content
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

fzf plugin fails to "find fzf installation directory" (because of changes in fzf) #12355

Closed
chfour opened this issue Apr 15, 2024 · 3 comments · Fixed by #12367
Closed

fzf plugin fails to "find fzf installation directory" (because of changes in fzf) #12355

chfour opened this issue Apr 15, 2024 · 3 comments · Fixed by #12367
Assignees
Labels
Area: plugin Issue or PR related to a plugin Feature New feature or request

Comments

@chfour
Copy link

chfour commented Apr 15, 2024

Describe the bug

I updated my nixos system yesterday and now after launching a shell I get the following message:

[oh-my-zsh] fzf plugin: Cannot find fzf installation directory.
Please add `export FZF_BASE=/path/to/fzf/install/dir` to your .zshrc

I tried specifying that env var, but after digging into the code it seems the plugin wants the completion.zsh and key-bindings.zsh files, which it can't find because as of fzf 0.48.0 they are included in the binary and returned concatenated together by fzf --zsh. I'm guessing the fix is to just eval "$(fzf --zsh)" as the changelog says, instead of trying to source the files?

Steps to reproduce

  1. Enable plugin fzf with the fzf package available in PATH as usual
  2. Reload or launch new shell
  3. See error message

Expected behavior

the fzf plugin working i guess? lol

Screenshots and recordings

No response

OS / Linux distribution

NixOS 24.05 (nixos-unstable)

Zsh version

5.9

Terminal emulator

GNOME Terminal

If using WSL on Windows, which version of WSL

None

Additional context

No response

@dsh2dsh
Copy link

dsh2dsh commented Apr 19, 2024

Same problem here. As a workaround I've created 2 files in ~/.fzf dir:

completion.zsh:

eval "$(fzf --zsh)"

and empty key-bindings.zsh.

@mcornella
Copy link
Member

Can you test whether #12367 resolves this? You can do so via omz pr test 12367. Thanks!

@mcornella mcornella self-assigned this Apr 19, 2024
@mcornella mcornella added Feature New feature or request Area: plugin Issue or PR related to a plugin labels Apr 19, 2024
@dsh2dsh
Copy link

dsh2dsh commented Apr 19, 2024

omz pr test 12367

Thank you @mcornella , it fixed my issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: plugin Issue or PR related to a plugin Feature New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants