From 80367c8db877e65ba7727c834fae8ce078295f0d Mon Sep 17 00:00:00 2001 From: Aidan Gauland Date: Sat, 30 Apr 2022 15:30:56 +1200 Subject: [PATCH] nixos/nextcloud: Remove confusing comment There is a comment above the invocation of 'nextcloud-occ app:enable', stating that the script should not fail if any of the apps cannot be enabled, but there is nothing in place to suppress errors. The app:enable command already continues installing the remaining apps when one fails to install, and we do not want to suppress errors in the setup script, so this just removes the comment about not failing. --- nixos/modules/services/web-apps/nextcloud.nix | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/nixos/modules/services/web-apps/nextcloud.nix b/nixos/modules/services/web-apps/nextcloud.nix index a4b886821ebf..fda151c2a4dc 100644 --- a/nixos/modules/services/web-apps/nextcloud.nix +++ b/nixos/modules/services/web-apps/nextcloud.nix @@ -811,7 +811,7 @@ in { ${occ}/bin/nextcloud-occ config:system:delete trusted_domains ${optionalString (cfg.extraAppsEnable && cfg.extraApps != { }) '' - # Try to enable apps (don't fail when one of them cannot be enabled , eg. due to incompatible version) + # Try to enable apps ${occ}/bin/nextcloud-occ app:enable ${concatStringsSep " " (attrNames cfg.extraApps)} ''}