[Question] dependency: installation or loading? about zplug

[babarot]

Thank you for your report.

I have a question about the dependency expressed with the pipe |. Does this represent the order in which plugins are installed or loaded (or both)?

I think it's both. Because this feature is from vim-plug.

nice tag in zplug certainly can represent the dependency of the plugins, but under normal circumstances, it should be used as the feature for loading after compinit (ref. #26). However, because I wasn't keen on treating nice tag as a special thing, I generalized it and then implemented. As a result, nice tag can also be used as representaion of dependency.

In short:

• The pipe should be used for dependency of the plugins
• Provided that it is necessary to decide specifications (installation/loading dependencies, or both)
• nice tag should be used for loading after compinit (More flexibly, can also be used as dependency management)

[NigoroJr]

Thanks, that makes sense. But then, would that mean the case above is a bug? The expected output of running it is:

plugin a
plugin b

because plugin a should be loaded before plugin b, right?

[babarot]

That is a bug just like you said.

[babarot]

In haste, we should fix that bug. Then, there is also a need to make sure the specifications.

[NigoroJr]

Yes, I was reading the source code for a while trying to figure out how the dependencies are set, but wasn't quite sure what is causing this 😢 It seems like the on tag is not set when plugin b is checked for its dependency but that's about as far as I got.

[babarot]

To tell the truth, the mechanism of managing dependencies is most complex section in zplug. Because the processes connected by a pipe is run in a subshell, all processing need to be run in the process.
https://github.com/b4b4r07/zplug/blob/feature/notifier/zplug#L1796-L1815

It seems like the on tag is not set when plugin b is checked for its dependency but that's about as far as I got.

Okay, I'll seek it.

[babarot]

That is a bug just like you said.

Fixed.

[NigoroJr]

Works perfectly now, thank you!