From 71a30f9e74141918e98a6bb6457dc97f158d988e Mon Sep 17 00:00:00 2001 From: Lily Foster Date: Fri, 16 Dec 2022 16:32:44 -0500 Subject: types: pass through mountOptions to _config for all types --- example/complex.nix | 2 +- example/luks-lvm.nix | 4 ++-- example/lvm-raid.nix | 2 +- example/zfs-over-legacy.nix | 2 +- types.nix | 20 +++++++++----------- 5 files changed, 14 insertions(+), 16 deletions(-) diff --git a/example/complex.nix b/example/complex.nix index 590dd9b..b361196 100644 --- a/example/complex.nix +++ b/example/complex.nix @@ -109,7 +109,7 @@ type = "filesystem"; format = "ext4"; mountpoint = "/ext4_on_lvm"; - options = [ + mountOptions = [ "defaults" ]; }; diff --git a/example/luks-lvm.nix b/example/luks-lvm.nix index af1c4a3..a6879ce 100644 --- a/example/luks-lvm.nix +++ b/example/luks-lvm.nix @@ -17,7 +17,7 @@ type = "filesystem"; format = "vfat"; mountpoint = "/boot"; - options = [ + mountOptions = [ "defaults" ]; }; @@ -52,7 +52,7 @@ type = "filesystem"; format = "ext4"; mountpoint = "/"; - options = [ + mountOptions = [ "defaults" ]; }; diff --git a/example/lvm-raid.nix b/example/lvm-raid.nix index 8622238..70ea197 100644 --- a/example/lvm-raid.nix +++ b/example/lvm-raid.nix @@ -89,7 +89,7 @@ type = "filesystem"; format = "ext4"; mountpoint = "/"; - options = [ + mountOptions = [ "defaults" ]; }; diff --git a/example/zfs-over-legacy.nix b/example/zfs-over-legacy.nix index 974af35..8e02b00 100644 --- a/example/zfs-over-legacy.nix +++ b/example/zfs-over-legacy.nix @@ -17,7 +17,7 @@ type = "filesystem"; format = "vfat"; mountpoint = "/boot"; - options = [ + mountOptions = [ "defaults" ]; }; diff --git a/types.nix b/types.nix index a65f2c1..91cc7c1 100644 --- a/types.nix +++ b/types.nix @@ -266,7 +266,7 @@ rec { }; mountOptions = mkOption { type = types.listOf types.str; - default = []; + default = [ "defaults" ]; }; _meta = mkOption { internal = true; @@ -323,7 +323,7 @@ rec { }; mountOptions = mkOption { type = types.listOf types.str; - default = []; + default = [ "defaults" ]; }; subvolumes = mkOption { type = types.listOf optionTypes.pathname; @@ -376,6 +376,7 @@ rec { fileSystems.${config.mountpoint} = { device = dev; fsType = "btrfs"; + options = config.mountOptions; }; }]; }; @@ -400,11 +401,7 @@ rec { }; mountOptions = mkOption { type = types.listOf types.str; - default = []; - }; - options = mkOption { - type = types.listOf types.str; - default = []; + default = [ "defaults" ]; }; mountpoint = mkOption { type = optionTypes.absolute-pathname; @@ -450,6 +447,7 @@ rec { fileSystems.${config.mountpoint} = { device = dev; fsType = config.format; + options = config.mountOptions; }; }]; }; @@ -932,7 +930,7 @@ rec { }; mountOptions = mkOption { type = types.listOf types.str; - default = []; + default = [ "defaults" ]; }; datasets = mkOption { type = types.attrsOf zfs_dataset; @@ -990,7 +988,7 @@ rec { fileSystems.${config.mountpoint} = { device = config.name; fsType = "zfs"; - options = lib.optional ((config.options.mountpoint or "") != "legacy") "zfsutil"; + options = config.mountOptions ++ lib.optional ((config.options.mountpoint or "") != "legacy") "zfsutil"; }; }) ]; @@ -1024,7 +1022,7 @@ rec { }; mountOptions = mkOption { type = types.listOf types.str; - default = []; + default = [ "defaults" ]; }; # filesystem options @@ -1086,7 +1084,7 @@ rec { fileSystems.${config.mountpoint} = { device = "${zpool}/${config.name}"; fsType = "zfs"; - options = lib.optional ((config.options.mountpoint or "") != "legacy") "zfsutil"; + options = config.mountOptions ++ lib.optional ((config.options.mountpoint or "") != "legacy") "zfsutil"; }; }); }; -- cgit v1.2.3