Skip to content

Conversation

@MattSturgeon
Copy link
Member

Inspired by home.version and home.enableNixpkgsReleaseCheck in Home Manager.
Print a warning when lib or pkgs are from a different release to Nixvim.

@khaneliman
Copy link
Contributor

              error: attribute 'lib' missing
              at /var/lib/buildbot-worker/worker-056/nix-community_nixvim_nix-eval/build/modules/misc/version.nix:51:23:
                  50|         libRelease = lib.trivial.release;
                  51|         pkgsRelease = pkgs.lib.trivial.release;
                    |                       ^
                  52|         releaseMismatch = nixvimRelease != libRelease || nixvimRelease != pkgsRelease;

Interesting...

@MattSturgeon
Copy link
Member Author

              error: attribute 'lib' missing
              at /var/lib/buildbot-worker/worker-056/nix-community_nixvim_nix-eval/build/modules/misc/version.nix:51:23:
                  50|         libRelease = lib.trivial.release;
                  51|         pkgsRelease = pkgs.lib.trivial.release;
                    |                       ^
                  52|         releaseMismatch = nixvimRelease != libRelease || nixvimRelease != pkgsRelease;

Interesting...

I think one of the tests uses a stubbed pkgs. I'll look into it a bit later.

Inspired by `home.version` and `home.enableNixpkgsReleaseCheck` in Home
Manager. Print a warning when `lib` or `pkgs` are from a different
release to Nixvim.
@MattSturgeon MattSturgeon force-pushed the version-check-warning branch from c4b81a9 to bd09e91 Compare December 3, 2025 06:46
@MattSturgeon MattSturgeon added this pull request to the merge queue Dec 3, 2025
Merged via the queue into nix-community:main with commit b0f3a36 Dec 3, 2025
4 checks passed
@MattSturgeon MattSturgeon deleted the version-check-warning branch December 3, 2025 09:30
@nixvim-ci
Copy link
Contributor

nixvim-ci bot commented Dec 3, 2025

Successfully created backport PR for nixos-25.11:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants