yadunut/clan-core
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

fix(auto-imports): disable since this is not needed anymore and causing collision with the new module type

Browse Source
This commit is contained in:
Johannes Kirschbauer
2025-03-29 16:24:48 +01:00
parent 64b7356d3f
commit 50cac33857
1 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
lib/build-clan/module.nix
View File

@@ -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 = _: {