From 547e101f1d6bf326d286bac0fb3272738a92a67f Mon Sep 17 00:00:00 2001 From: Junegunn Choi Date: Wed, 12 Jul 2023 13:55:59 +0900 Subject: [PATCH] Use $SHELL instead of bash if it's known to support 'pipefail' when running the default find command Close #3339 Close #3364 --- src/reader.go | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/src/reader.go b/src/reader.go index 06e9b73..494a2f7 100644 --- a/src/reader.go +++ b/src/reader.go @@ -6,6 +6,7 @@ import ( "io" "os" "os/exec" + "path" "path/filepath" "sync" "sync/atomic" @@ -98,8 +99,17 @@ func (r *Reader) ReadSource() { r.startEventPoller() var success bool if util.IsTty() { - // The default command for *nix requires bash + // The default command for *nix requires a shell that supports "pipefail" + // https://unix.stackexchange.com/a/654932/62171 shell := "bash" + currentShell := os.Getenv("SHELL") + currentShellName := path.Base(currentShell) + for _, shellName := range []string{"bash", "zsh", "ksh", "ash", "hush", "mksh", "yash"} { + if currentShellName == shellName { + shell = currentShell + break + } + } cmd := os.Getenv("FZF_DEFAULT_COMMAND") if len(cmd) == 0 { if defaultCommand != "" {