Skip to content

[master] flake.lock: Update#8923

Merged
khaneliman merged 3 commits into
masterfrom
update/master
Mar 19, 2026
Merged

[master] flake.lock: Update#8923
khaneliman merged 3 commits into
masterfrom
update/master

Conversation

@home-manager-ci
Copy link
Copy Markdown
Contributor

@home-manager-ci home-manager-ci Bot commented Mar 18, 2026

Automated update by the update-flake-lock GitHub Action.

Flake lock file updates:

• Updated input 'nixpkgs':
    'github:NixOS/nixpkgs/c06b4ae3d6599a672a6210b7021d699c351eebda?narHash=sha256-wvfdLLWJ2I9oEpDd9PfMA8osfIZicoQ5MT1jIwNs9Tk%3D' (2026-03-13)
  → 'github:NixOS/nixpkgs/b40629efe5d6ec48dd1efba650c797ddbd39ace0?narHash=sha256-TJ3lSQtW0E2JrznGVm8hOQGVpXjJyXY2guAxku2O9A4%3D' (2026-03-18)

This PR was most recently updated by workflow run 23315011266.

@home-manager-ci home-manager-ci Bot added the dependencies Pull requests that update a dependency file label Mar 18, 2026
@teto
Copy link
Copy Markdown
Collaborator

teto commented Mar 19, 2026

nix run .#tests -- neovim work for nixos-unstable but not make test. Turns out one is using nixos-unstable while the other is using nixpkgs-unstable and currently nixpkgs-unstable is older than nixos-unstable.
This is confusing and we should use a single source of truth. I would favor nixos-unstable since:

  • if you are on nixos, you are more likely to use nixos-unstable than nixpkgs-unstable
  • the allegedly advantage of nixpkgs-unstable being more up to date is not necessarily true as showcased here.

@khaneliman
Copy link
Copy Markdown
Collaborator

khaneliman commented Mar 19, 2026

the allegedly advantage of nixpkgs-unstable being more up to date is not necessarily true as showcased here.

It's extremely rare for it to not be the case, but it does happen.

If you are on nixos, you are more likely to use nixos-unstable than nixpkgs-unstable

We don't make that assertion, we support anywhere nix can be run.

Much rather track nixpkgs-unstable for home level confguration but I think flake.nix is nixos-unstable atm and we should match.

Flake lock file updates:

• Updated input 'nixpkgs':
    'github:NixOS/nixpkgs/c06b4ae3d6599a672a6210b7021d699c351eebda?narHash=sha256-wvfdLLWJ2I9oEpDd9PfMA8osfIZicoQ5MT1jIwNs9Tk%3D' (2026-03-13)
  → 'github:NixOS/nixpkgs/b40629efe5d6ec48dd1efba650c797ddbd39ace0?narHash=sha256-TJ3lSQtW0E2JrznGVm8hOQGVpXjJyXY2guAxku2O9A4%3D' (2026-03-18)
@khaneliman
Copy link
Copy Markdown
Collaborator

khaneliman commented Mar 19, 2026

@teto looks like neovim breaking change from upstream. Needs update for module injecting config, test assertion, or update to module to use legacyWrapper also affected nixvim nix-community/nixvim#4219

@teto
Copy link
Copy Markdown
Collaborator

teto commented Mar 19, 2026

yeah, that's what #8734 is for I dont merge in nixpkgs before testing my PR against HM. How shall we proceed ? can I push here or we merge this one than the fixes ?

@khaneliman
Copy link
Copy Markdown
Collaborator

yeah, that's what #8734 is for I dont merge in nixpkgs before testing my PR against HM. How shall we proceed ? can I push here or we merge this one than the fixes ?

Ideally, we don't introduce broken behavior in a flake update. I wait for green CI so a push here would be fine.

now that home-manager always generates an init.lua we can take the
wrapper output instead of reimplemnting stuff in HM (notwithstanding rtp
changes that are not needed in HM).
This is necessary to keep the provider configuration available after
this change NixOS/nixpkgs#487390

Also updated test to reflect ruby disabling
@teto
Copy link
Copy Markdown
Collaborator

teto commented Mar 19, 2026

I think the neovim commit fixes the neovim test. The remaining failure I am not familiar with. Seems like a keepass dependency not building on aarch64

Signed-off-by: Austin Horstman <khaneliman12@gmail.com>
@khaneliman khaneliman merged commit 9d3c1d6 into master Mar 19, 2026
8 checks passed
@khaneliman khaneliman deleted the update/master branch March 19, 2026 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file editors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants