-
Notifications
You must be signed in to change notification settings - Fork 134
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[bug]: Bad math expression in zinit plugins
on Ubuntu-22.04/WSL2
#545
Comments
Seeing the same in a similar environment (Windows 11, WSL2, Ubuntu 22.04, zsh 5.8.1), but with more plugins installed.
|
I think there are some extra double quotes that are not correct inside (( "ZINIT[STATES__$i]" )) && is_loaded="{happy}L" The above (current) version produces the error. It works fine after changing it to this: (( ZINIT[STATES__$i] )) && is_loaded="{happy}L" |
Good catch. I wonder why this issue doesn't occur on macOS...? If you have time, please test the bug fix via branch |
@vladdoster @pschmitt what happened with this change on the main branch? It looks like there was a merge, then an attempted revert, but instead I know how to easily fix this locally, but it might not be as obvious to everyone. |
What happened?
I found an error in the
zinit plugins
command on WSL2/Ubuntu-22.04/zsh-5.8.1 combo.This is a fresh install,
.zshrc
only has zinit's installation and initialization (no plugins at all)This does not happen on MacOS with zsh 5.9 but replicated on another machine with same setup.
Steps to reproduce
Just install zsh from apt in a fresh Ubuntu 22.04 on WSL2 on Windows 10
Create a .zshrc containing only the following:
Start zsh and wait until zinit is installed
Run
zi plugins
Relevant output
The text was updated successfully, but these errors were encountered: