Repository navigation
chore(auto): update flake inputs - #1803
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI repair report
What I changed
Nothing.
nix run .#world -- check(as captured in.ci-check.log, andre-verified by re-running it) evaluates every
nixosConfigurations,homeConfigurations,devShellsandpackagesoutput forx86_64-linuxsuccessfully (all ✅, no ❌, no eval errors, noassertion failures). This flake-input bump is a small, same-week
nixpkgs/nixpkgs-stablebump (about 8 hours apart) plus routine bumpsof
devenv,fenix,emacs-overlay,helix,nixos-hardware,noctalia-shell,NUR,rust-overlay,wezterm,zen-browser-flakeand
claude-code-nix— seeflake.lock. I searched the repo for everyoption/attribute named in the warnings below (
stdenv.isLinux,stdenv.isDarwin,devenv-up,devenv-test,options.json,rekey.hostPubkey) and none of them are set or referenced in thisrepo's own
.nixfiles, so there is no local option to rename or pin.Deprecation warnings surfaced by the update
All four evaluation warnings in
.ci-check.logtrace to code insidethird-party flake inputs, not to any option this repo sets:
stdenv.isLinux is deprecated, use stdenv.hostPlatform.isLinux instead/ the matchingstdenv.isDarwinwarning — emitted whileevaluating the
niripackage graph (pulled in viausers/profiles/niri.nixand friends), from inside a transitivedependency's own package expression (fetched via
nix-filter, notvendored in this repo). Grepped the whole tree for
stdenv.isLinux/stdenv.isDarwin: no matches outside the Nix store.Nothing here to rename.
The package 'devenv-up' is deprecated. Use the corresponding \devenv ` commands.and the matchingdevenv-testwarning — these two packages are generated automatically byinputs.devenv.flakeModule(wired up inflake/devenv.nix) for every devenv shell underdevenv/. The deprecation notice is baked into devenv's own flake module output, not into anything this repo configures; we never referencedevenv-up/devenv-testby name ourselves,nix flake check` just evaluates every package the moduleexposes.
Using 'builtins.derivation' to create a derivation named 'options.json' ... without a proper context— not associated withany file in this repo (no
options.json,nixos-render-docs, orbuildOptionsJSONreference anywhere in the tree); it comes from anoptions-doc derivation inside one of the flake inputs.
None of these map to a NixOS/home-manager/nix-darwin module option that
this repo sets, so there's no rename-with-same-effect to apply, and per
the "boring fix" rule there's nothing safe to pin either — pinning
would mean patching someone else's flake input, which is a bigger,
riskier change than the warning justifies.
The two
evaluation warning: You have not yet specified rekey.hostPubkey for your host test/...host nixosmessages areagenix-rekey's normal first-deploy notice for the
testandinstallerhosts (both intentionally have nohostPubkeyset yet, perthe warning's own text: "This is intentional so you can initially
deploy your system to read the actual pubkey"). This is pre-existing
behaviour, not something the flake update changed, and touches secrets
handling, so it's covered by "Needs a human" below regardless.
Needs a human
rekey.hostPubkeyunset fortestandinstallerhosts.Pre-existing, secrets-related, and intentional per the warning text —
not something to "fix" without knowing whether those hosts are meant
to stay in first-deploy state. Flagging per the "secrets/key material"
exclusion rather than changing anything.
changes were introduced by this input bump.
Verified with
nix run .#world -- check.