home: ssh: add missing 'mkIf cfg.enable'

Introduce the recursive version of this function, then refactor to
reduce repetition.

home/mail/msmtp.nix

@@ -1,4 +1,4 @@
-{ config, lib, pkgs, ... }:
+{ config, lib, ... }:
 let
   cfg = config.my.home.mail.msmtp;
 in

home/ssh.nix

@@ -7,7 +7,7 @@ in
     enable = mkDisableOption "ssh configuration";
   };
 
-  config.programs.ssh = {
+  config.programs.ssh = lib.mkIf cfg.enable {
     enable = true;
 
     matchBlocks = {

lib/default.nix

@@ -12,8 +12,7 @@ let
   };
 
   mylib = makeExtensible (self:
-    with self; mapModules ./.
+    mapModules ./. (file: import file { inherit self lib pkgs inputs; })
-      (file: import file { inherit self lib pkgs inputs; })
   );
 in
 mylib.extend (self: super:

lib/modules.nix

@@ -3,21 +3,48 @@ let
   inherit (builtins) readDir pathExists;
   inherit (lib) hasPrefix hasSuffix nameValuePair removeSuffix;
   inherit (self.attrs) mapFilterAttrs;
-in
+
-{
+  implOptionalRecursion = recurse:
-  mapModules = dir: fn:
+    let
+      recurseStep =
+        if recurse
+        then (n: path: fn: nameValuePair n (impl path fn))
+        else (_: _: _: nameValuePair "" null);
+      impl = dir: fn:
         mapFilterAttrs
+          (n: _: n != "" && !(hasPrefix "_" n))
           (n: v:
-        v != null &&
+            let
-        !(hasPrefix "_" n))
+              path = "${toString dir}/${n}";
-      (n: v:
+            in
-        let path = "${toString dir}/${n}"; in
+            if v == "directory"
-        if v == "directory" && pathExists "${path}/default.nix"
+            then
+              if pathExists "${path}/default.nix"
               then nameValuePair n (fn path)
-        else if v == "regular" &&
+              else recurseStep n path fn
-          n != "default.nix" &&
+            else if v == "regular" && n != "default.nix" && hasSuffix ".nix" n
-          hasSuffix ".nix" n
             then nameValuePair (removeSuffix ".nix" n) (fn path)
             else nameValuePair "" null)
           (readDir dir);
+    in
+    impl;
+in
+{
+  # Find all nix modules in a directory, discard any prefixed with "_",
+  # map a function to each resulting path, and generate an attribute set
+  # to associate module name to resulting value.
+  #
+  # mapModules ::
+  #   path
+  #   (path -> any)
+  #   attrs
+  mapModules = implOptionalRecursion false;
+
+  # Recursive version of mapModules.
+  #
+  # mapModulesRec ::
+  #   path
+  #   (path -> any)
+  #   attrs
+  mapModulesRec = implOptionalRecursion true;
 }

machines/aramis/default.nix

@@ -2,7 +2,7 @@
 # your system.  Help is available in the configuration.nix(5) man page
 # and in the NixOS manual (accessible by running ‘nixos-help’).
 
-{ config, pkgs, ... }:
+{ ... }:
 
 {
   imports = [

machines/aramis/services.nix

@@ -1,4 +1,4 @@
-{ lib, ... }:
+{ ... }:
 {
   config.my.services = {
     wireguard = {

modules/hardware/upower.nix

@@ -1,4 +1,4 @@
-{ config, lib, pkgs, ... }:
+{ config, lib, ... }:
 let
   cfg = config.my.hardware.upower;
 in

modules/services/paperless.nix

@@ -1,4 +1,4 @@
-{ config, lib, pkgs, ... }:
+{ config, lib, ... }:
 let
   cfg = config.my.services.paperless;
 in

modules/services/podgrab.nix

@@ -1,5 +1,5 @@
 # A simple podcast fetcher
-{ config, lib, pkgs, ... }:
+{ config, lib, ... }:
 let
   cfg = config.my.services.podgrab;
 in

pkgs/nolimips/default.nix

@@ -1,4 +1,4 @@
-{ lib, fetchurl, gnulib, stdenv }:
+{ lib, fetchurl, stdenv }:
 stdenv.mkDerivation rec {
   pname = "nolimips";
   version = "0.11";

pkgs/unbound-zones-adblock/default.nix

@@ -1,4 +1,4 @@
-{ lib, fetchFromGitHub, gawk, stdenvNoCC, unified-hosts-lists }:
+{ lib, gawk, stdenvNoCC, unified-hosts-lists }:
 stdenvNoCC.mkDerivation rec {
   name = "unbound-zones-adblock";
   version = unified-hosts-lists.version;