From ba2cb8fa40becbd0b6451512d9662751729b2488 Mon Sep 17 00:00:00 2001 From: Kiara Grouwstra Date: Fri, 11 Jul 2025 16:04:34 +0200 Subject: [PATCH] add note on better solution --- infra/common/proxmox-qemu-vm.nix | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/infra/common/proxmox-qemu-vm.nix b/infra/common/proxmox-qemu-vm.nix index 7ce22dbf..961bb963 100644 --- a/infra/common/proxmox-qemu-vm.nix +++ b/infra/common/proxmox-qemu-vm.nix @@ -1,5 +1,13 @@ let - # pulling this in manually over from module args resolves an infinite recursion + # pulling this in manually over from module args resolves an infinite recursion. + # FIXME: instead untangle `//infra/flake-part.nix` and make it stop passing wild functions. + # move moving towards a portable-services-like pattern where some things are submodules. + # Right now those wild functions are for parameterising a bunch of things, + # and the modular way to do that would be options -- + # obviously you can't use those for `imports`, + # so one way to decouple fixpoints is to isolate them into submodules. + # Therefore one approach would be to try to go down the call graph, + # and see where what's currently a function could be a `submodule` field of something else. sources = import ../../npins; in {