diff options
| author |  Tom van der Lee <tom@vanderlee.io> | 2026-09-18 14:58:29 +0200 |
|---|---|---|
| committer | Tom van der Lee <tom@vanderlee.io> | 2026-09-18 14:58:29 +0200 |
| commit | a6d5680c87c31ab6da8dcb7cdeb76e77a02924e1 (patch) | |
| tree | d5740741ebc9289241bbfc913b52d9150c7ebb62 /openspec | |
| parent | 414777f8f9ee0bb5ca501709c6df5a3497ec77ba (diff) | |
| download | desktop-flake-a6d5680c87c31ab6da8dcb7cdeb76e77a02924e1.tar.gz desktop-flake-a6d5680c87c31ab6da8dcb7cdeb76e77a02924e1.tar.bz2 desktop-flake-a6d5680c87c31ab6da8dcb7cdeb76e77a02924e1.zip | |
Fix invalid home-manager.useUserHomeDirectory option in mkNixos
home-manager.useUserHomeDirectory was never a real option -- a mistake
that slipped through because this file was never actually evaluated
until a live nixos-rebuild switch surfaced it today. Corrected to
useUserPackages, the standard option that pairs with useGlobalPkgs for
NixOS+home-manager integration (also what Nix's own error suggested).
Diffstat (limited to 'openspec')
| -rw-r--r-- | openspec/changes/add-nix-flake-bootstrap/tasks.md | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/openspec/changes/add-nix-flake-bootstrap/tasks.md b/openspec/changes/add-nix-flake-bootstrap/tasks.md index 47d476f..853a5be 100644 --- a/openspec/changes/add-nix-flake-bootstrap/tasks.md +++ b/openspec/changes/add-nix-flake-bootstrap/tasks.md | |||
| @@ -5,7 +5,7 @@ | |||
| 5 | 5 | ||
| 6 | ## 2. Shared lib helpers | 6 | ## 2. Shared lib helpers |
| 7 | 7 | ||
| 8 | - [x] 2.1 Create `lib/mkNixos.nix` wrapping `nixpkgs.lib.nixosSystem`, taking `{ hostname, system ? "x86_64-linux", users ? [ "tom" ], extraModules ? [] }`, including `modules/nixos/common.nix`, `../hosts/${hostname}`, and `home-manager.nixosModules.home-manager` with `home-manager.users.<u> = import ../users/${u}.nix` for each user, and `specialArgs = { inherit inputs; }` — verify by importing it in a `nix repl` and checking it returns a function (**deferred, no local nix**) | 8 | - [x] 2.1 Create `lib/mkNixos.nix` wrapping `nixpkgs.lib.nixosSystem`, taking `{ hostname, system ? "x86_64-linux", users ? [ "tom" ], extraModules ? [] }`, including `modules/nixos/common.nix`, `../hosts/${hostname}`, and `home-manager.nixosModules.home-manager` with `home-manager.users.<u> = import ../users/${u}.nix` for each user, and `specialArgs = { inherit inputs; }` — verify by importing it in a `nix repl` and checking it returns a function (**deferred, no local nix**) (**correction, 2026-09-18: a real `nixos-rebuild switch` on an actual host surfaced that this file used a non-existent option, `home-manager.useUserHomeDirectory` — the manual review above missed it since it was never actually evaluated until now. Fixed to the correct, standard option `home-manager.useUserPackages` (which Nix's own error message suggested). This is the first concrete evidence that "verified by manual review" was not a substitute for real evaluation — flagging for awareness on any other task in this repo marked complete under the same no-local-Nix constraint.**) |
| 9 | - [x] 2.2 Create `lib/mkHome.nix` wrapping `home-manager.lib.homeManagerConfiguration`, taking `{ username, system, extraModules ? [] }`, using `pkgs = nixpkgs.legacyPackages.${system}` and `modules = [ ../users/${username}.nix ] ++ extraModules` — verify the same way (**deferred, no local nix**) | 9 | - [x] 2.2 Create `lib/mkHome.nix` wrapping `home-manager.lib.homeManagerConfiguration`, taking `{ username, system, extraModules ? [] }`, using `pkgs = nixpkgs.legacyPackages.${system}` and `modules = [ ../users/${username}.nix ] ++ extraModules` — verify the same way (**deferred, no local nix**) |
| 10 | - [x] 2.3 Create `lib/default.nix` aggregating both as `{ mkNixos, mkHome }` given `{ inputs }` — verify `flake.nix` can `import ./lib { inherit inputs; }` without eval errors (**deferred, no local nix**) | 10 | - [x] 2.3 Create `lib/default.nix` aggregating both as `{ mkNixos, mkHome }` given `{ inputs }` — verify `flake.nix` can `import ./lib { inherit inputs; }` without eval errors (**deferred, no local nix**) |
| 11 | 11 | ||