vimUtils: deprecate configure.pathogen (#154814)

pathogen does not bring any value compared to vim native packages so
remove it to ease maintainance burden.
This commit is contained in:
Matthieu Coudron 2022-06-13 15:25:46 +02:00 committed by GitHub
parent 5ed2235050
commit a1ad235743
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
6 changed files with 18 additions and 24 deletions

View File

@ -277,9 +277,6 @@ my-vim =
copy paste output2 here copy paste output2 here
]; ];
# Pathogen would be
# vimrcConfig.pathogen.knownPlugins = plugins; # plugins
# vimrcConfig.pathogen.pluginNames = ["tlib"];
}; };
``` ```

View File

@ -121,6 +121,14 @@
this version for the entire lifecycle of the 22.11 release. this version for the entire lifecycle of the 22.11 release.
</para> </para>
</listitem> </listitem>
<listitem>
<para>
(Neo)Vim can not be configured with
<literal>configure.pathogen</literal> anymore to reduce
maintainance burden. Use <literal>configure.packages</literal>
instead.
</para>
</listitem>
</itemizedlist> </itemizedlist>
</section> </section>
<section xml:id="sec-release-22.11-notable-changes"> <section xml:id="sec-release-22.11-notable-changes">

View File

@ -50,6 +50,9 @@ In addition to numerous new and upgraded packages, this release has the followin
- PHP 7.4 is no longer supported due to upstream not supporting this - PHP 7.4 is no longer supported due to upstream not supporting this
version for the entire lifecycle of the 22.11 release. version for the entire lifecycle of the 22.11 release.
- (Neo)Vim can not be configured with `configure.pathogen` anymore to reduce maintainance burden.
Use `configure.packages` instead.
<!-- To avoid merge conflicts, consider adding your item at an arbitrary place in the list instead. --> <!-- To avoid merge conflicts, consider adding your item at an arbitrary place in the list instead. -->
## Other Notable Changes {#sec-release-22.11-notable-changes} ## Other Notable Changes {#sec-release-22.11-notable-changes}

View File

@ -150,8 +150,8 @@ rec {
# and do `:call deoplete#enable()`. It will print an error if the remote # and do `:call deoplete#enable()`. It will print an error if the remote
# plugin is not registered. # plugin is not registered.
test_nvim_with_remote_plugin = neovim.override { test_nvim_with_remote_plugin = neovim.override {
extraName = "-pathogen-remote"; extraName = "-remote";
configure.pathogen.pluginNames = with vimPlugins; [ deoplete-nvim ]; configure.packages.foo.start = with vimPlugins; [ deoplete-nvim ];
}; };
# only neovim makes use of `requiredPlugins`, test this here # only neovim makes use of `requiredPlugins`, test this here

View File

@ -51,9 +51,6 @@ vim-with-plugins in PATH:
# full documentation at github.com/MarcWeber/vim-addon-manager # full documentation at github.com/MarcWeber/vim-addon-manager
]; ];
# there is a pathogen implementation as well, but its startup is slower and [VAM] has more feature
# vimrcConfig.pathogen.knownPlugins = vimPlugins; # optional
# vimrcConfig.pathogen.pluginNames = ["vim-addon-nix"];
}; };
WHAT IS A VIM PLUGIN? WHAT IS A VIM PLUGIN?
@ -103,7 +100,7 @@ It might happen than a plugin is not known by vim-pi yet. We encourage you to
contribute to vim-pi so that plugins can be updated automatically. contribute to vim-pi so that plugins can be updated automatically.
CREATING DERVITATIONS AUTOMATICALLY BY PLUGIN NAME CREATING DERIVATIONS AUTOMATICALLY BY PLUGIN NAME
================================================== ==================================================
Most convenient is to use a ~/.vim-scripts file putting a plugin name into each line Most convenient is to use a ~/.vim-scripts file putting a plugin name into each line
as documented by [VAM]'s README.md as documented by [VAM]'s README.md
@ -277,24 +274,18 @@ let
}: }:
let let
/* pathogen mostly can set &rtp at startup time. Its used very commonly. /* pathogen mostly can set &rtp at startup time. Deprecated.
*/ */
pathogenImpl = let pathogenImpl = let
knownPlugins = pathogen.knownPlugins or vimPlugins; knownPlugins = pathogen.knownPlugins or vimPlugins;
plugins = findDependenciesRecursively (map (pluginToDrv knownPlugins) pathogen.pluginNames); plugins = findDependenciesRecursively (map (pluginToDrv knownPlugins) pathogen.pluginNames);
pluginsEnv = buildEnv { pathogenPackages.pathogen = lib.warn "'pathogen' attribute is deprecated. Use 'packages' instead in your vim configuration" {
name = "pathogen-plugin-env"; start = plugins;
paths = map (x: "${x}/${rtpPath}") plugins;
}; };
in in
'' nativeImpl pathogenPackages;
let &rtp.=(empty(&rtp)?"":',')."${vimPlugins.vim-pathogen.rtp}"
execute pathogen#infect('${pluginsEnv}/{}')
filetype indent plugin on | syn on
'';
/* vim-plug is an extremely popular vim plugin manager. /* vim-plug is an extremely popular vim plugin manager.
*/ */

View File

@ -23,11 +23,6 @@ in
vimrcConfig.vam.pluginDictionaries = [{name = "vim-nix"; }]; vimrcConfig.vam.pluginDictionaries = [{name = "vim-nix"; }];
}; };
test_vim_with_vim_nix_using_pathogen = vim_configurable.customize {
name = "vim-with-vim-addon-nix-using-pathogen";
vimrcConfig.pathogen.pluginNames = [ "vim-nix" ];
};
test_vim_with_vim_nix_using_plug = vim_configurable.customize { test_vim_with_vim_nix_using_plug = vim_configurable.customize {
name = "vim-with-vim-addon-nix-using-plug"; name = "vim-with-vim-addon-nix-using-plug";
vimrcConfig.plug.plugins = with vimPlugins; [ vim-nix ]; vimrcConfig.plug.plugins = with vimPlugins; [ vim-nix ];