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 0.39.0 missing runtime dependency bc #225579

Closed
DanielSiepmann opened this issue Apr 10, 2023 · 5 comments · Fixed by #237365
Closed

fzf 0.39.0 missing runtime dependency bc #225579

DanielSiepmann opened this issue Apr 10, 2023 · 5 comments · Fixed by #237365

Comments

@DanielSiepmann
Copy link
Contributor

DanielSiepmann commented Apr 10, 2023

Describe the bug

Installing and using fzf breaks with the following error message:

.nix-profile/bin/fzf-tmux: line 183: bc: command not found

Steps To Reproduce

Steps to reproduce the behavior:

  1. Install fzf with tmux integration, I did it via home manager modules
  2. Call fzf with tmux integration within tmux

Expected behavior

The tmux pop up appears with history to filter

Screenshots

If applicable, add screenshots to help explain your problem.

Additional context

Add any other context about the problem here.

Notify maintainers

@Br1ght0ne @Ma27 @zowoq

Metadata

Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.105, NixOS, 23.05 (Stoat), 23.05pre470969.0e19daa510e`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `"nixos, nixos-hardware, phps"`
 - channels(daniels): `"home-manager, nixpkgs, phps"`
 - nixpkgs: `/home/daniels/.nix-defexpr/channels/nixpkgs`

Looks like this got introduced with #224377. The runtime dependency was added with that new version: junegunn/fzf@0.38.0...0.39.0#diff-29747b0804ea04e1d230bd8651260a6b8687374cc1ae4c2d70ec26f05c4f719bR183 Line 183 of file bin/fzf-tmux I wasn't able to find a proper URL to link to that line within the diff.

@DanielSiepmann
Copy link
Contributor Author

Workaround: Install bc as another package.

@DanielSiepmann
Copy link
Contributor Author

@msfjarvis due to providing the update to 0.39.0 at #224377

msfjarvis added a commit to msfjarvis/nixpkgs that referenced this issue Jun 12, 2023
@msfjarvis
Copy link
Contributor

Should be fixed by #237365, would be helpful if you could test it.

@DanielSiepmann
Copy link
Contributor Author

Thanks @msfjarvis. I'm pretty new, is there any doc how I could test this on my machine?

@msfjarvis
Copy link
Contributor

Thanks @msfjarvis. I'm pretty new, is there any doc how I could test this on my machine?

If you're using Nix channels then you can replace your nixpkgs channel with https://github.com/msfjarvis/nixpkgs/archive/refs/heads/hs/fix-fzf-tmux.tar.gz and rebuild, which will build all your packages from my PR branch, including fzf.

If you're using Nix Flakes then you can change the nixpkgs input to be github:msfjarvis/nixpkgs/hs/fix-fzf-tmux which has the same effect.

msfjarvis added a commit to msfjarvis/nixpkgs that referenced this issue Jun 13, 2023
msfjarvis added a commit to msfjarvis/nixpkgs that referenced this issue Jun 19, 2023
msfjarvis added a commit to msfjarvis/nixpkgs that referenced this issue Jun 19, 2023
msfjarvis added a commit to msfjarvis/nixpkgs that referenced this issue Jun 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants