Commit 7a875ea1 authored by Matthieu C.'s avatar Matthieu C. Committed by Matthieu Coudron
Browse files

vimPlugins: add warning for nvimSkipModule deprecation 

It is best effort since this was not part of a release yet.
I tried to do display the warning at eval time but it ended up being harder than expected.
I tried to rewrite buildVimPlugin with lib.extendMkDerivation to access the fixed point to no avail.
I think it is something worth doing but in a separate PR.

Here is a quickfix to avoid it ending up in release.
parent 3e6edb83

pkgs/applications/editors/vim/plugins/hooks/neovim-require-check-hook.sh

+6 −0
Original line number Diff line number Diff line
@@ -54,6 +54,12 @@ run_require_checks() { 
    local nativeCheckInputs="${nativeBuildInputs[*]}"

    local checkInputs="${buildInputs[*]}"

    set +e



    if [ -v 'nvimSkipModule' ]; then

        nvimSkipModules=("${nvimSkipModule[@]}")

        echo "WARNING: nvimSkipModule got renamed to nvimSkipModules, please update package $name"

    fi



    for name in "${nvimRequireCheck[@]}"; do

        local skip=false

        for module in "${nvimSkipModules[@]}"; do