caveats: remove ZSH FPATH logic.

This is too error-prone and full of edge cases.

Fixes #9455
This commit is contained in:
Mike McQuaid 2020-12-09 11:47:48 +00:00
parent 37fe150284
commit 1f8fd2a9ef
No known key found for this signature in database
GPG Key ID: 48A898132FD8EE70

View File

@ -128,21 +128,10 @@ class Caveats
#{root_dir}/etc/bash_completion.d
EOS
when :zsh
site_functions = root_dir/"share/zsh/site-functions"
zsh_caveats = +<<~EOS
<<~EOS
zsh #{installed.join(" and ")} have been installed to:
#{site_functions}
#{root_dir}/share/zsh/site-functions
EOS
zsh = which("zsh") || which("zsh", ENV["HOMEBREW_PATH"])
if zsh.present? && Utils.popen_read("'#{zsh}' -ic 'echo $FPATH'").exclude?(site_functions.to_s)
zsh_caveats << <<~EOS
#{site_functions} is not in your zsh FPATH!
Add it by following these steps:
#{Formatter.url("https://docs.brew.sh/Shell-Completion#configuring-completions-in-zsh")}
EOS
end
zsh_caveats.freeze
when :fish
fish_caveats = +"fish #{installed.join(" and ")} have been installed to:"
fish_caveats << "\n #{root_dir}/share/fish/vendor_completions.d" if completion_installed