zap-zsh / zap

:zap: Zap is a minimal zsh plugin manager
https://www.zapzsh.com/
GNU General Public License v3.0
942 stars 33 forks source link

install.sh: ZSH(<5.8): main:zparseopts:17: no default array defined: -F #169

Closed mar4th3 closed 1 year ago

mar4th3 commented 1 year ago

Describe the bug

Running install.zsh on older versions of zsh (<5.8) throws the following error

main:zparseopts:17: no default array defined: -F

My knowledge of zsh is very limitied so I am unable to propose a principled fix.

Removing the -F options seems to work for me.

Here a similar issue (w/ possible fix)

Steps to reproduce

Expected behavior

Instalation should go finish w/o errors

Screenshots and recordings

No response

OS / Linux distribution

openSUSE Leap (15.3)

Zsh version

5.6

Zap version

release-v1O

Terminal emulator

kitty

If using WSL on Windows, which version of WSL

None

Additional context

No response

wintermi commented 1 year ago

Thanks @mar4th3 for raising this issue. A pull request has been submitted with the fix.

wintermi commented 1 year ago

Pull Request merged, please reopen this issue should you continue to have a problem with the install.