Seirdy / term-dmenu

Replace dmenu with a floating terminal and FZF! Comes with helper scripts for program/app launching, window switching, etc.
https://gitlab.com/Seirdy/term-dmenu
Other
50 stars 1 forks source link

term-dmenu evaluates output #2

Open calbrecht opened 2 years ago

calbrecht commented 2 years ago

This is a stripped down version with abduco and stderr redirects removed, but basically the same as term-dmenu.

export input='$(echo hi, i get evaluated)'; dash -c 'output=$(echo "$input" | fzf); export output; dash -c "echo \"$output\""'

I noticed this behaviour when using rg output as input to fzf in term-dmenu.

RayZ0rr commented 2 years ago

Yes, I was looking for this fix. It would also be cool to make j4-dmenu-desktop optional in the sense that it'll be used if it exists.

RayZ0rr commented 2 years ago

@calbrecht Would it also be possible to use bash instead of dash? Bash is in pretty much all systems I use so if possible I would like to avoid installing dash.