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
vimPlugins: Fix update.py, add/update plugins #46496
Conversation
@@ -292,7 +292,7 @@ def generate_nix(plugins: List[Tuple[str, Plugin]]): | |||
name = "{plugin.normalized_name}-{plugin.version}"; | |||
src = fetchFromGitHub {{ | |||
owner = "{owner}"; | |||
repo = "{plugin.name}"; | |||
repo = "{repo}"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, snap. I wonder why I was able to build all plugins. I blame fixed-input derivations for that.
@@ -2,16 +2,6 @@ | |||
{ buildVimPluginFrom2Nix, fetchFromGitHub }: | |||
|
|||
{ | |||
a-vim = buildVimPluginFrom2Nix { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you changed this manually?
@@ -182,8 +182,8 @@ | |||
}; | |||
}; | |||
|
|||
Colour-Sampler-Pack = buildVimPluginFrom2Nix { | |||
name = "Colour-Sampler-Pack-2012-11-30"; | |||
colorsamplerpack = buildVimPluginFrom2Nix { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and this one.
I would prefer if we do not manually have to change stuff in this file. |
After running the |
Ah it get its name by a reverse lookup from the aliases. |
This pull request fixes that: #46561 |
Motivation for this change
The
update.py
script in latestmaster
was generating wronggenerated.nix
- therepo
attr would have.
replaced by-
.(becauseplugin.name
contains the name with.
replaced by-
). Refs #46394Note, that
easymotion
vim plugin has been moved to easymotion/easymotioncc @Mic92
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)