From d2aaeac42c563cf8cf76ee3b90d9585dadbb91e0 Mon Sep 17 00:00:00 2001 From: Robert Helgesson Date: Sun, 30 May 2021 11:54:33 +0200 Subject: [PATCH] files: remove assertion on recursive onChange See discussion in https://github.com/nix-community/home-manager/pull/2031 --- modules/files.nix | 11 ----------- modules/lib/file-type.nix | 4 ++-- 2 files changed, 2 insertions(+), 13 deletions(-) diff --git a/modules/files.nix b/modules/files.nix index 9eb4fd05e..320cab73c 100644 --- a/modules/files.nix +++ b/modules/files.nix @@ -39,17 +39,6 @@ in }; config = { - assertions = [( - let - conflicts = mapAttrsToList (n: v: n) - (filterAttrs (n: v: v.recursive && v.onChange != "") cfg); - in { - assertion = conflicts == []; - message = '' - Cannot use 'home.file..onChange' when 'home.file..recursive' is enabled: - ${concatStringsSep ", " conflicts}''; - })]; - lib.file.mkOutOfStoreSymlink = path: let pathStr = toString path; diff --git a/modules/lib/file-type.nix b/modules/lib/file-type.nix index d0ae1f31d..71babc79a 100644 --- a/modules/lib/file-type.nix +++ b/modules/lib/file-type.nix @@ -83,8 +83,8 @@ with lib; after the new files have been linked into place. - Note, this option cannot be used when recursive - is enabled. + Note, this code is always run when recursive is + enabled. ''; };