diff options
author | Daniel Schaefer <git@danielschaefer.me> | 2018-05-05 15:41:28 +0700 |
---|---|---|
committer | Daniel Schaefer <git@danielschaefer.me> | 2018-05-05 15:55:53 +0700 |
commit | 43650115e34ea610e53f718d9e02de7222ac0ba2 (patch) | |
tree | 95a7ca5e78e329bd5db392e07776e65d7277e0b5 /nixos/modules/misc | |
parent | 34a71a80b8fb74cb07b7239845a667172e0d374f (diff) | |
download | nixpkgs-43650115e34ea610e53f718d9e02de7222ac0ba2.tar nixpkgs-43650115e34ea610e53f718d9e02de7222ac0ba2.tar.gz nixpkgs-43650115e34ea610e53f718d9e02de7222ac0ba2.tar.bz2 nixpkgs-43650115e34ea610e53f718d9e02de7222ac0ba2.tar.lz nixpkgs-43650115e34ea610e53f718d9e02de7222ac0ba2.tar.xz nixpkgs-43650115e34ea610e53f718d9e02de7222ac0ba2.tar.zst nixpkgs-43650115e34ea610e53f718d9e02de7222ac0ba2.zip |
nixos/documentation: Correct use of lib.optional
lib.optional returns a singleton or an empty list. Therefore the argument does not need to be wrapped in a list. An alternative patch could have used lib.optionals but seems like no more elements are going to be added to the optional list.
Diffstat (limited to 'nixos/modules/misc')
-rw-r--r-- | nixos/modules/misc/documentation.nix | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/nixos/modules/misc/documentation.nix b/nixos/modules/misc/documentation.nix index 2e426c01708..b482a5a6752 100644 --- a/nixos/modules/misc/documentation.nix +++ b/nixos/modules/misc/documentation.nix @@ -75,20 +75,20 @@ let cfg = config.documentation; in (mkIf cfg.man.enable { environment.systemPackages = [ pkgs.man-db ]; environment.pathsToLink = [ "/share/man" ]; - environment.extraOutputsToInstall = [ "man" ] ++ optional cfg.dev.enable [ "devman" ]; + environment.extraOutputsToInstall = [ "man" ] ++ optional cfg.dev.enable "devman"; }) (mkIf cfg.info.enable { environment.systemPackages = [ pkgs.texinfoInteractive ]; environment.pathsToLink = [ "/share/info" ]; - environment.extraOutputsToInstall = [ "info" ] ++ optional cfg.dev.enable [ "devinfo" ]; + environment.extraOutputsToInstall = [ "info" ] ++ optional cfg.dev.enable "devinfo"; }) (mkIf cfg.doc.enable { # TODO(@oxij): put it here and remove from profiles? # environment.systemPackages = [ pkgs.w3m ]; # w3m-nox? environment.pathsToLink = [ "/share/doc" ]; - environment.extraOutputsToInstall = [ "doc" ] ++ optional cfg.dev.enable [ "devdoc" ]; + environment.extraOutputsToInstall = [ "doc" ] ++ optional cfg.dev.enable "devdoc"; }) ]); |