Skip to content

Fix buildVimPlugin#933

Merged
saghen merged 2 commits intosaghen:mainfrom
konradmalik:fix-require-check
Jan 7, 2025
Merged

Fix buildVimPlugin#933
saghen merged 2 commits intosaghen:mainfrom
konradmalik:fix-require-check

Conversation

@konradmalik
Copy link
Contributor

@konradmalik konradmalik commented Jan 7, 2025

After NixOS/nixpkgs#352277 when we update nixpkgs in this repo - the plugin build fails with:

       > Require check failed for the following modules:
       >   - repro

See: https://github.com/Saghen/blink.cmp/actions/runs/12654863130/job/35263764079?pr=933

Since this file is not needed for any release, we should simply ignore it from nvim sources instead of providing explicit require checks.

What we have here is:

  • update flake
  • ignore repro.lua from nvimFs (also ignore docs folder because it's quite large and not used at all in vimPlugin)

The "update" step is optional, I just wanted to confirm that the build will fail and that this will fix it. If updating now is not preferred, we can remove that commit.

Note - we could also use nvimSkipModule to ignore repro but I think we can just exclude it from source. However I'm open to suggestions.

Flake lock file updates:

• Updated input 'nixpkgs':
    'github:nixos/nixpkgs/4989a246d7a390a859852baddb1013f825435cee?narHash=sha256-kMBQ5PRiFLagltK0sH%2B08aiNt3zGERC2297iB6vrvlU%3D' (2024-12-17)
  → 'github:nixos/nixpkgs/b30f97d8c32d804d2d832ee837d0f1ca0695faa5?narHash=sha256-U47xeACNBpkSO6IcCm0XvahsVXpJXzjPIQG7TZlOToU%3D' (2025-01-06)
@konradmalik konradmalik marked this pull request as ready for review January 7, 2025 16:15
@konradmalik konradmalik changed the title Fix nvim Fix buildVimPlugin Jan 7, 2025
@saghen saghen merged commit 3f5dcbc into saghen:main Jan 7, 2025
@saghen
Copy link
Owner

saghen commented Jan 7, 2025

Thanks!

@konradmalik konradmalik deleted the fix-require-check branch January 8, 2025 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants