-
Notifications
You must be signed in to change notification settings - Fork 128
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]: zinit should check that all persisted ices are defined during update #540
Comments
Ok, found the problem and it's me: the problem is that the above update line does not register the annex (as the
fixes the problem for me... |
Ok, one could argue that any not registered ices should error out even during update, but I guess that's a hard problem to solve :-( |
Did you try running zsh via zsh --interactive --login -c -- or possibly zinit via zsh -ils -c -- '@zinit-scheduler burst' |
I did not, but I'm also fine with running the extra The fix would be to also check that all the ices are defined when running update and the ices are loaded from the respective plugin folder and not given via the cli. -> Update the initial message and title. |
What happened?
During
zinit update
, the ices are loaded from a file in the plugin folder. These ices seems to not get validated and if some ice are not defined, they are silently ignored.zinit update
should validate all ice which are loaded from the persisted file during update.Original issue which showed this problem
I've the following recipe:
I usually run the following command to update all zinit stuff automatically from a bigger script:
This currently breaks the dl parts of the fzf recipe which in turn breaks the src, resulting in
Steps to reproduce
Relevant output
(slightly shortend the progress bars)
-> Note that the dl lines are not in the second version
Operating System & Version
OS: darwin21.3.0 | Vendor: apple | Machine: x86_64 | CPU: x86_64 | Processor: i386 | Hardware: x86_64
(this also happens on my linux machone, so it shouldn't be OS specific)
Zsh version
zsh 5.9 (x86_64-apple-darwin21.3.0)
Terminal emulator
wezterm
Code of Conduct
The text was updated successfully, but these errors were encountered: