tree ec99a557943656ae07dc3e8d464fc14bde7f344c
parent 3637b8aa1063963ea91fa2cf9cb8813c62e231d3
author Skyler Grey <sky@a.starrysky.fyi> 1722628024 +0000
committer Skyler Grey <minion@clicks.codes> 1722632457 +0000

fix(non-clicks): Re-export dependencies

Due to how mkIf continues to check definitions past a `false`, this is
invalid unless `environment.persistence` is defined

config = lib.mkIf false {
  environment.persistence = {
    ...
  };
};

This is a big problem for people importing our modules, who may not have
had, say, `impermanence` imported

We considered using `lib.optionalAttrs` instead, but this causes a lot
of infinite recursion elsewhere in our config, so we decided this was a
more "cost effective" way of spending our time, even if it will likely
cause downstream configs to import more than they technically need...

Change-Id: I433d34395b0449de17af2dea2a8a3138c1ff6dba
Reviewed-on: https://git.clicks.codes/c/Infra/NixFiles/+/808
Reviewed-by: Skyler Grey <minion@clicks.codes>
Tested-by: Skyler Grey <minion@clicks.codes>