-
Notifications
You must be signed in to change notification settings - Fork 24
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
Reasoning behind "symlink" and "copy" strategy #316
Comments
I didn't think about cloning repositories to Vim 9Vim script code is parsed on every executions, real interpreter language in 2020. I don't know it really makes uglifying useless, but at least, I am also thinking that all repositories should be in
and I thought it in the context of #172. Want to skip
|
Hi,
this is just a question.
What is the reason for downloading the actual git repository to
$VOLTREPOSPATH
and later copying or symlinking them to~/.vim/pack/volt/opt
instead of storing them in
~/.vim/pack/volt/opt
directly?Since the
bundled_plugconf.vim
file is generated for the current profile,the
~/.vim/pack/volt/opt
directory could actually contain the repositoriesfor all profiles, but the
bundled_plugconf.vim
file would only load thosethat are active in the current profile.
This would also allow the user to activate a plugin manually that is not
contained in the current profile.¹
Is there something I a missing? Is there some additional benefit of
separating the
$VOLTREPOSPATH
and the~/.vim/pack/volt/opt
directory?I thought about adding an additional build strategy to volt for the purpose
of downloading the git repositories directly to
~/.vim/pack/volt/opt
, butnoticed that this would be an intrusive change that needed to be applied to a
bunch of other places, too.
[¹] However, that would leave out the plugconf file for that plugin. To
allow reading the plugconf file in such a case the user would need a
vim-volt plugin that provides a command (e.g. Packadd) to load the package
with packadd and read in the corresponding plugconf file.
The text was updated successfully, but these errors were encountered: