[bash,zsh] Fix the version check for mawk

[akinomyoga]

This fixes #4570. See the commit message for details.

[akinomyoga]

@LangLangBart @LimingFang Could you check the behavior of this patch?

[LimingFang]

It seems there's still sth wrong🤔

[LangLangBart]

@LangLangBart @LimingFang Could you check the behavior of this patch?

tested with debian 10, the error no longer shows up.

It seems there's still sth wrong🤔

can u share the output from typeset -f __fzf_exec_awk

I believe you need to either compile the fzf binary or source the patched file

[LimingFang]

Output of typeset -f __fzf_exec_awk is:

__fzf_exec_awk () {
	if [[ -z ${__fzf_awk-} ]]
	then
		__fzf_awk=awk
		if [[ $OSTYPE == solaris* && -x /usr/xpg4/bin/awk ]]
		then
			__fzf_awk=/usr/xpg4/bin/awk
		elif command -v mawk > /dev/null 2>&1
		then
			local n x y z d
			IFS=' .' read -r n x y z d <<< $(command mawk -W version 2> /dev/null)
			[[ $n == mawk ]] && ((d >= 20230302 && (x * 1000 + y) * 1000 + z >= 1003004)) && __fzf_awk=mawk
		fi
	fi
	LC_ALL=C exec "$__fzf_awk" "$@"
}

I just remove my fzf repo and use branch in @akinomyoga 's repo and ./install.
Debian10 too

[akinomyoga]

I just remove my fzf repo and use branch in @akinomyoga 's repo and ./install.

Unfortunately, ./install just downloads the latest pre-built binary from the GitHub Release page, so it just installs 0.66.1 regardless of the current working tree of the repository.

[LimingFang]

I just remove my fzf repo and use branch in @akinomyoga 's repo and ./install.

Unfortunately, ./install just downloads the latest pre-built binary from the GitHub Release page, so it just installs 0.66.1 regardless of the current working tree of the repository.

My bad, everything works fine now. cc @LangLangBart

[akinomyoga]

Thanks!

[junegunn]

Thanks!

[akinomyoga]

Thank you!