From 48d910f11fe8912d0224af3fbed97c335883885a Mon Sep 17 00:00:00 2001 From: Johannes Kirschbauer Date: Sat, 29 Mar 2025 16:24:48 +0100 Subject: [PATCH] fix(auto-imports): disable since this is not needed anymore and causing collision with the new module type --- lib/build-clan/module.nix | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/lib/build-clan/module.nix b/lib/build-clan/module.nix index 17e828608..0117ce42e 100644 --- a/lib/build-clan/module.nix +++ b/lib/build-clan/module.nix @@ -165,7 +165,18 @@ let in { imports = [ - ./auto-imports.nix + # Temporarily disable auto-imports since the type of the modules is not a plain path anymore we cant "merge" multiple definitions + # That this feature worked previously seems like a coincidence. + # TODO(@Qubasa): make sure modules are not imported twice. + # Example error: + # The option `inventory.modules.admin' is defined multiple times while it's expected to be unique. + # - In `/nix/store/a0iqxl7r1spqsf2b886kn3i5sj8p37nc-source/lib/build-clan/auto-imports.nix': /nix/store/a0iqxl7r1spqsf2b886kn3i5sj8p37nc-source/clanModules/admin + # - In `/nix/store/a0iqxl7r1spqsf2b886kn3i5sj8p37nc-source/lib/build-clan/module.nix': /nix/store/a0iqxl7r1spqsf2b886kn3i5sj8p37nc-source/clanModules/admin + # + # After the inventory refactoring we might not need this anymore + # People can just import the module they want to use: `module = { input = "inputName"; name = "moduleName"; };` + # ./auto-imports.nix + # Merge the inventory file { inventory = _: {