laloch / xontrib-fzf-widgets

Set of fzf widgets for xonsh
GNU General Public License v3.0
33 stars 18 forks source link

fix: in TMUX context, fzf (not fzf-tmux) is found. #24

Open ylmrx opened 2 years ago

ylmrx commented 2 years ago

we have :

$ def get_fzf_binary_name():
.     fzf_tmux_cmd = 'fzf-tmux'
.     if 'TMUX' in ${...} and $(which fzf_tmux_cmd):
.         return fzf_tmux_cmd
.
.     return 'fzf'
.                 
[11:02:21]  fuzzy@fricadin ~  [no k8s context]
$ get_fzf_binary_name() 
fzf_tmux_cmd not in $PATH or xonsh.builtins.aliases
'fzf'

we want :

$ def get_fzf_binary_name():
.     fzf_tmux_cmd = 'fzf-tmux'
.     if 'TMUX' in ${...} and $(which @(fzf_tmux_cmd)):
.         return fzf_tmux_cmd
.
.     return 'fzf'
.                 
[10:59:39]  fuzzy@fricadin ~  [no k8s context]
$ get_fzf_binary_name()
'fzf-tmux'