treewide: use more lib.optionalString

Changed files
+89 -95
nixos
modules
i18n
input-method
installer
cd-dvd
programs
security
services
audio
backup
blockchain
ethereum
continuous-integration
jenkins
games
logging
mail
misc
monitoring
prometheus
exporters
network-filesystems
openafs
networking
security
system
cachix-agent
web-apps
web-servers
system
boot
loader
grub
systemd-boot
tasks
virtualisation
+1 -4
nixos/modules/i18n/input-method/ibus.nix
···
check = x: (lib.types.package.check x) && (attrByPath ["meta" "isIbusEngine"] false x);
};
-
impanel =
-
if cfg.panel != null
-
then "--panel=${cfg.panel}"
-
else "";
+
impanel = optionalString (cfg.panel != null) "--panel=${cfg.panel}";
ibusAutostart = pkgs.writeTextFile {
name = "autostart-ibus-daemon";
+2 -2
nixos/modules/installer/cd-dvd/iso-image.nix
···
(option: ''
menuentry '${defaults.name} ${
# Name appended to menuentry defaults to params if no specific name given.
-
option.name or (if option ? params then "(${option.params})" else "")
-
}' ${if option ? class then " --class ${option.class}" else ""} {
+
option.name or (optionalString (option ? params) "(${option.params})")
+
}' ${optionalString (option ? class) " --class ${option.class}"} {
linux ${defaults.image} \''${isoboot} ${defaults.params} ${
option.params or ""
}
+1 -1
nixos/modules/programs/less.nix
···
${concatStringsSep "\n"
(mapAttrsToList (command: action: "${command} ${action}") cfg.commands)
}
-
${if cfg.clearDefaultCommands then "#stop" else ""}
+
${optionalString cfg.clearDefaultCommands "#stop"}
#line-edit
${concatStringsSep "\n"
+8 -8
nixos/modules/programs/tmux.nix
···
{ config, pkgs, lib, ... }:
let
-
inherit (lib) mkOption mkIf types;
+
inherit (lib) mkOption mkIf types optionalString;
cfg = config.programs.tmux;
···
set -g base-index ${toString cfg.baseIndex}
setw -g pane-base-index ${toString cfg.baseIndex}
-
${if cfg.newSession then "new-session" else ""}
+
${optionalString cfg.newSession "new-session"}
-
${if cfg.reverseSplit then ''
+
${optionalString cfg.reverseSplit ''
bind v split-window -h
bind s split-window -v
-
'' else ""}
+
''}
set -g status-keys ${cfg.keyMode}
set -g mode-keys ${cfg.keyMode}
-
${if cfg.keyMode == "vi" && cfg.customPaneNavigationAndResize then ''
+
${optionalString (cfg.keyMode == "vi" && cfg.customPaneNavigationAndResize) ''
bind h select-pane -L
bind j select-pane -D
bind k select-pane -U
···
bind -r J resize-pane -D ${toString cfg.resizeAmount}
bind -r K resize-pane -U ${toString cfg.resizeAmount}
bind -r L resize-pane -R ${toString cfg.resizeAmount}
-
'' else ""}
+
''}
-
${if (cfg.shortcut != defaultShortcut) then ''
+
${optionalString (cfg.shortcut != defaultShortcut) ''
# rebind main key: C-${cfg.shortcut}
unbind C-${defaultShortcut}
set -g prefix C-${cfg.shortcut}
bind ${cfg.shortcut} send-prefix
bind C-${cfg.shortcut} last-window
-
'' else ""}
+
''}
setw -g aggressive-resize ${boolToStr cfg.aggressiveResize}
setw -g clock-mode-style ${if cfg.clock24 then "24" else "12"}
+2 -2
nixos/modules/security/acme/default.nix
···
# FIXME Most of these custom warnings and filters for security.acme.certs.* are required
# because using mkRemovedOptionModule/mkChangedOptionModule with attrsets isn't possible.
-
warnings = filter (w: w != "") (mapAttrsToList (cert: data: if data.extraDomains != "_mkMergedOptionModule" then ''
+
warnings = filter (w: w != "") (mapAttrsToList (cert: data: optionalString (data.extraDomains != "_mkMergedOptionModule") ''
The option definition `security.acme.certs.${cert}.extraDomains` has changed
to `security.acme.certs.${cert}.extraDomainNames` and is now a list of strings.
Setting a custom webroot for extra domains is not possible, instead use separate certs.
-
'' else "") cfg.certs);
+
'') cfg.certs);
assertions = let
certs = attrValues cfg.certs;
+2 -2
nixos/modules/services/audio/snapserver.nix
···
warnings =
# https://github.com/badaix/snapcast/blob/98ac8b2fb7305084376607b59173ce4097c620d8/server/streamreader/stream_manager.cpp#L85
-
filter (w: w != "") (mapAttrsToList (k: v: if v.type == "spotify" then ''
+
filter (w: w != "") (mapAttrsToList (k: v: optionalString (v.type == "spotify") ''
services.snapserver.streams.${k}.type = "spotify" is deprecated, use services.snapserver.streams.${k}.type = "librespot" instead.
-
'' else "") cfg.streams);
+
'') cfg.streams);
systemd.services.snapserver = {
after = [ "network.target" ];
+1 -1
nixos/modules/services/backup/mysql-backup.nix
···
'';
backupDatabaseScript = db: ''
dest="${cfg.location}/${db}.gz"
-
if ${mariadb}/bin/mysqldump ${if cfg.singleTransaction then "--single-transaction" else ""} ${db} | ${gzip}/bin/gzip -c > $dest.tmp; then
+
if ${mariadb}/bin/mysqldump ${optionalString cfg.singleTransaction "--single-transaction"} ${db} | ${gzip}/bin/gzip -c > $dest.tmp; then
mv $dest.tmp $dest
echo "Backed up to $dest"
else
+1 -1
nixos/modules/services/backup/restic.nix
···
filesFromTmpFile = "/run/restic-backups-${name}/includes";
backupPaths =
if (backup.dynamicFilesFrom == null)
-
then if (backup.paths != null) then concatStringsSep " " backup.paths else ""
+
then optionalString (backup.paths != null) (concatStringsSep " " backup.paths)
else "--files-from ${filesFromTmpFile}";
pruneCmd = optionals (builtins.length backup.pruneOpts > 0) [
(resticCmd + " forget --prune " + (concatStringsSep " " backup.pruneOpts))
+2 -2
nixos/modules/services/blockchain/ethereum/geth.nix
···
--gcmode ${cfg.gcmode} \
--port ${toString cfg.port} \
--maxpeers ${toString cfg.maxpeers} \
-
${if cfg.http.enable then ''--http --http.addr ${cfg.http.address} --http.port ${toString cfg.http.port}'' else ""} \
+
${optionalString cfg.http.enable ''--http --http.addr ${cfg.http.address} --http.port ${toString cfg.http.port}''} \
${optionalString (cfg.http.apis != null) ''--http.api ${lib.concatStringsSep "," cfg.http.apis}''} \
-
${if cfg.websocket.enable then ''--ws --ws.addr ${cfg.websocket.address} --ws.port ${toString cfg.websocket.port}'' else ""} \
+
${optionalString cfg.websocket.enable ''--ws --ws.addr ${cfg.websocket.address} --ws.port ${toString cfg.websocket.port}''} \
${optionalString (cfg.websocket.apis != null) ''--ws.api ${lib.concatStringsSep "," cfg.websocket.apis}''} \
${optionalString cfg.metrics.enable ''--metrics --metrics.addr ${cfg.metrics.address} --metrics.port ${toString cfg.metrics.port}''} \
--authrpc.addr ${cfg.authrpc.address} --authrpc.port ${toString cfg.authrpc.port} --authrpc.vhosts ${lib.concatStringsSep "," cfg.authrpc.vhosts} \
+1 -1
nixos/modules/services/continuous-integration/jenkins/job-builder.nix
···
jobdir="${jenkinsCfg.home}/$jenkinsjobname"
rm -rf "$jobdir"
done
-
'' + (if cfg.accessUser != "" then reloadScript else "");
+
'' + (optionalString (cfg.accessUser != "") reloadScript);
serviceConfig = {
Type = "oneshot";
User = jenkinsCfg.user;
+1 -1
nixos/modules/services/games/minetest-server.nix
···
let
cfg = config.services.minetest-server;
-
flag = val: name: if val != null then "--${name} ${toString val} " else "";
+
flag = val: name: optionalString (val != null) "--${name} ${toString val} ";
flags = [
(flag cfg.gameId "gameid")
(flag cfg.world "world")
+2 -3
nixos/modules/services/logging/logrotate.nix
···
};
mailOption =
-
if foldr (n: a: a || (n.mail or false) != false) false (attrValues cfg.settings)
-
then "--mail=${pkgs.mailutils}/bin/mail"
-
else "";
+
optionalString (foldr (n: a: a || (n.mail or false) != false) false (attrValues cfg.settings))
+
"--mail=${pkgs.mailutils}/bin/mail";
in
{
imports = [
+1 -1
nixos/modules/services/logging/syslogd.nix
···
cfg = config.services.syslogd;
syslogConf = pkgs.writeText "syslog.conf" ''
-
${if (cfg.tty != "") then "kern.warning;*.err;authpriv.none /dev/${cfg.tty}" else ""}
+
${optionalString (cfg.tty != "") "kern.warning;*.err;authpriv.none /dev/${cfg.tty}"}
${cfg.defaultConfig}
${cfg.extraConfig}
'';
+1 -1
nixos/modules/services/mail/postfix.nix
···
headerChecks = concatStringsSep "\n" (map (x: "${x.pattern} ${x.action}") cfg.headerChecks) + cfg.extraHeaderChecks;
-
aliases = let separator = if cfg.aliasMapType == "hash" then ":" else ""; in
+
aliases = let separator = optionalString (cfg.aliasMapType == "hash") ":"; in
optionalString (cfg.postmasterAlias != "") ''
postmaster${separator} ${cfg.postmasterAlias}
''
+5 -5
nixos/modules/services/misc/gammu-smsd.nix
···
Connection = ${cfg.device.connection}
SynchronizeTime = ${if cfg.device.synchronizeTime then "yes" else "no"}
LogFormat = ${cfg.log.format}
-
${if (cfg.device.pin != null) then "PIN = ${cfg.device.pin}" else ""}
+
${optionalString (cfg.device.pin != null) "PIN = ${cfg.device.pin}"}
${cfg.extraConfig.gammu}
···
${optionalString (cfg.backend.service == "sql" && cfg.backend.sql.driver == "native_pgsql") (
with cfg.backend; ''
Driver = ${sql.driver}
-
${if (sql.database!= null) then "Database = ${sql.database}" else ""}
-
${if (sql.host != null) then "Host = ${sql.host}" else ""}
-
${if (sql.user != null) then "User = ${sql.user}" else ""}
-
${if (sql.password != null) then "Password = ${sql.password}" else ""}
+
${optionalString (sql.database!= null) "Database = ${sql.database}"}
+
${optionalString (sql.host != null) "Host = ${sql.host}"}
+
${optionalString (sql.user != null) "User = ${sql.user}"}
+
${optionalString (sql.password != null) "Password = ${sql.password}"}
'')}
${cfg.extraConfig.smsd}
+1 -1
nixos/modules/services/misc/gitlab.nix
···
enableDelete = true; # This must be true, otherwise GitLab won't manage it correctly
extraConfig = {
auth.token = {
-
realm = "http${if cfg.https == true then "s" else ""}://${cfg.host}/jwt/auth";
+
realm = "http${optionalString (cfg.https == true) "s"}://${cfg.host}/jwt/auth";
service = cfg.registry.serviceName;
issuer = cfg.registry.issuer;
rootcertbundle = cfg.registry.certFile;
+1 -1
nixos/modules/services/misc/mbpfan.nix
···
let
cfg = config.services.mbpfan;
-
verbose = if cfg.verbose then "v" else "";
+
verbose = optionalString cfg.verbose "v";
settingsFormat = pkgs.formats.ini {};
settingsFile = settingsFormat.generate "mbpfan.ini" cfg.settings;
+7 -7
nixos/modules/services/misc/redmine.nix
···
services.redmine.settings = {
production = {
-
scm_subversion_command = if cfg.components.subversion then "${pkgs.subversion}/bin/svn" else "";
-
scm_mercurial_command = if cfg.components.mercurial then "${pkgs.mercurial}/bin/hg" else "";
-
scm_git_command = if cfg.components.git then "${pkgs.git}/bin/git" else "";
-
scm_cvs_command = if cfg.components.cvs then "${pkgs.cvs}/bin/cvs" else "";
-
scm_bazaar_command = if cfg.components.breezy then "${pkgs.breezy}/bin/bzr" else "";
-
imagemagick_convert_command = if cfg.components.imagemagick then "${pkgs.imagemagick}/bin/convert" else "";
-
gs_command = if cfg.components.ghostscript then "${pkgs.ghostscript}/bin/gs" else "";
+
scm_subversion_command = optionalString cfg.components.subversion "${pkgs.subversion}/bin/svn";
+
scm_mercurial_command = optionalString cfg.components.mercurial "${pkgs.mercurial}/bin/hg";
+
scm_git_command = optionalString cfg.components.git "${pkgs.git}/bin/git";
+
scm_cvs_command = optionalString cfg.components.cvs "${pkgs.cvs}/bin/cvs";
+
scm_bazaar_command = optionalString cfg.components.breezy "${pkgs.breezy}/bin/bzr";
+
imagemagick_convert_command = optionalString cfg.components.imagemagick "${pkgs.imagemagick}/bin/convert";
+
gs_command = optionalString cfg.components.ghostscript "${pkgs.ghostscript}/bin/gs";
minimagick_font_path = "${cfg.components.minimagick_font_path}";
};
};
+1 -1
nixos/modules/services/misc/siproxd.nix
···
${optionalString (cfg.hostsAllowReg != []) "hosts_allow_reg = ${concatStringsSep "," cfg.hostsAllowReg}"}
${optionalString (cfg.hostsAllowSip != []) "hosts_allow_sip = ${concatStringsSep "," cfg.hostsAllowSip}"}
${optionalString (cfg.hostsDenySip != []) "hosts_deny_sip = ${concatStringsSep "," cfg.hostsDenySip}"}
-
${if (cfg.passwordFile != "") then "proxy_auth_pwfile = ${cfg.passwordFile}" else ""}
+
${optionalString (cfg.passwordFile != "") "proxy_auth_pwfile = ${cfg.passwordFile}"}
${cfg.extraConfig}
'';
+2 -2
nixos/modules/services/monitoring/prometheus/exporters/collectd.nix
···
};
};
serviceOpts = let
-
collectSettingsArgs = if (cfg.collectdBinary.enable) then ''
+
collectSettingsArgs = optionalString (cfg.collectdBinary.enable) ''
--collectd.listen-address ${cfg.collectdBinary.listenAddress}:${toString cfg.collectdBinary.port} \
--collectd.security-level ${cfg.collectdBinary.securityLevel} \
-
'' else "";
+
'';
in {
serviceConfig = {
ExecStart = ''
+2 -2
nixos/modules/services/network-filesystems/openafs/lib.nix
···
{ config, lib, ...}:
let
-
inherit (lib) concatStringsSep mkOption types;
+
inherit (lib) concatStringsSep mkOption types optionalString;
in {
mkCellServDB = cellName: db: ''
>${cellName}
-
'' + (concatStringsSep "\n" (map (dbm: if (dbm.ip != "" && dbm.dnsname != "") then dbm.ip + " #" + dbm.dnsname else "")
+
'' + (concatStringsSep "\n" (map (dbm: optionalString (dbm.ip != "" && dbm.dnsname != "") "${dbm.ip} #${dbm.dnsname}")
db))
+ "\n";
+1 -1
nixos/modules/services/networking/ndppd.nix
···
ttl ${toString proxy.ttl}
${render proxy.rules (ruleNetworkName: rule: ''
rule ${prefer rule.network ruleNetworkName} {
-
${rule.method}${if rule.method == "iface" then " ${rule.interface}" else ""}
+
${rule.method}${optionalString (rule.method == "iface") " ${rule.interface}"}
}'')}
}'')}
'');
+1 -1
nixos/modules/services/networking/ntopng.nix
···
redis.createInstance = mkOption {
type = types.nullOr types.str;
-
default = if versionAtLeast config.system.stateVersion "22.05" then "ntopng" else "";
+
default = optionalString (versionAtLeast config.system.stateVersion "22.05") "ntopng";
description = lib.mdDoc ''
Local Redis instance name. Set to `null` to disable
local Redis instance. Defaults to `""` for
+2 -2
nixos/modules/services/networking/ssh/lshd.nix
···
else (concatStrings (map (i: "--interface=\"${i}\"")
interfaces))} \
-h "${hostKey}" \
-
${if !syslog then "--no-syslog" else ""} \
+
${optionalString (!syslog) "--no-syslog" } \
${if passwordAuthentication then "--password" else "--no-password" } \
${if publicKeyAuthentication then "--publickey" else "--no-publickey" } \
${if rootLogin then "--root-login" else "--no-root-login" } \
-
${if loginShell != null then "--login-shell=\"${loginShell}\"" else "" } \
+
${optionalString (loginShell != null) "--login-shell=\"${loginShell}\"" } \
${if srpKeyExchange then "--srp-keyexchange" else "--no-srp-keyexchange" } \
${if !tcpForwarding then "--no-tcpip-forward" else "--tcpip-forward"} \
${if x11Forwarding then "--x11-forward" else "--no-x11-forward" } \
+5 -5
nixos/modules/services/networking/ssh/sshd.nix
···
mkdir -m 0755 -p "$(dirname '${k.path}')"
ssh-keygen \
-t "${k.type}" \
-
${if k ? bits then "-b ${toString k.bits}" else ""} \
-
${if k ? rounds then "-a ${toString k.rounds}" else ""} \
-
${if k ? comment then "-C '${k.comment}'" else ""} \
-
${if k ? openSSHFormat && k.openSSHFormat then "-o" else ""} \
+
${optionalString (k ? bits) "-b ${toString k.bits}"} \
+
${optionalString (k ? rounds) "-a ${toString k.rounds}"} \
+
${optionalString (k ? comment) "-C '${k.comment}'"} \
+
${optionalString (k ? openSSHFormat && k.openSSHFormat) "-o"} \
-f "${k.path}" \
-N ""
fi
···
'') cfg.ports}
${concatMapStrings ({ port, addr, ... }: ''
-
ListenAddress ${addr}${if port != null then ":" + toString port else ""}
+
ListenAddress ${addr}${optionalString (port != null) (":" + toString port)}
'') cfg.listenAddresses}
${optionalString cfgc.setXAuthLocation ''
+3 -3
nixos/modules/services/networking/strongswan.nix
···
inherit (builtins) toFile;
inherit (lib) concatMapStringsSep concatStringsSep mapAttrsToList
-
mkIf mkEnableOption mkOption types literalExpression;
+
mkIf mkEnableOption mkOption types literalExpression optionalString;
cfg = config.services.strongswan;
···
strongswanConf = {setup, connections, ca, secretsFile, managePlugins, enabledPlugins}: toFile "strongswan.conf" ''
charon {
-
${if managePlugins then "load_modular = no" else ""}
-
${if managePlugins then ("load = " + (concatStringsSep " " enabledPlugins)) else ""}
+
${optionalString managePlugins "load_modular = no"}
+
${optionalString managePlugins ("load = " + (concatStringsSep " " enabledPlugins))}
plugins {
stroke {
secrets_file = ${secretsFile}
+2 -2
nixos/modules/services/networking/stunnel.nix
···
environment.systemPackages = [ pkgs.stunnel ];
environment.etc."stunnel.cfg".text = ''
-
${ if cfg.user != null then "setuid = ${cfg.user}" else "" }
-
${ if cfg.group != null then "setgid = ${cfg.group}" else "" }
+
${ optionalString (cfg.user != null) "setuid = ${cfg.user}" }
+
${ optionalString (cfg.group != null) "setgid = ${cfg.group}" }
debug = ${cfg.logLevel}
+1 -1
nixos/modules/services/networking/xinetd.nix
···
${optionalString srv.unlisted "type = UNLISTED"}
${optionalString (srv.flags != "") "flags = ${srv.flags}"}
socket_type = ${if srv.protocol == "udp" then "dgram" else "stream"}
-
${if srv.port != 0 then "port = ${toString srv.port}" else ""}
+
${optionalString (srv.port != 0) "port = ${toString srv.port}"}
wait = ${if srv.protocol == "udp" then "yes" else "no"}
user = ${srv.user}
server = ${srv.server}
+2 -3
nixos/modules/services/security/oauth2_proxy.nix
···
} // (getProviderOptions cfg cfg.provider) // cfg.extraConfig;
mapConfig = key: attr:
-
if attr != null && attr != [] then (
+
optionalString (attr != null && attr != []) (
if isDerivation attr then mapConfig key (toString attr) else
if (builtins.typeOf attr) == "set" then concatStringsSep " "
(mapAttrsToList (name: value: mapConfig (key + "-" + name) value) attr) else
if (builtins.typeOf attr) == "list" then concatMapStringsSep " " (mapConfig key) attr else
if (builtins.typeOf attr) == "bool" then "--${key}=${boolToString attr}" else
if (builtins.typeOf attr) == "string" then "--${key}='${attr}'" else
-
"--${key}=${toString attr}")
-
else "";
+
"--${key}=${toString attr}");
configString = concatStringsSep " " (mapAttrsToList mapConfig allConfig);
in
+1 -1
nixos/modules/services/system/cachix-agent/default.nix
···
EnvironmentFile = cfg.credentialsFile;
ExecStart = ''
${cfg.package}/bin/cachix ${lib.optionalString cfg.verbose "--verbose"} ${lib.optionalString (cfg.host != null) "--host ${cfg.host}"} \
-
deploy agent ${cfg.name} ${if cfg.profile != null then cfg.profile else ""}
+
deploy agent ${cfg.name} ${optionalString (cfg.profile != null) cfg.profile}
'';
};
};
+2 -2
nixos/modules/services/web-apps/discourse.nix
···
services.postfix = lib.mkIf cfg.mail.incoming.enable {
enable = true;
-
sslCert = if cfg.sslCertificate != null then cfg.sslCertificate else "";
-
sslKey = if cfg.sslCertificateKey != null then cfg.sslCertificateKey else "";
+
sslCert = lib.optionalString (cfg.sslCertificate != null) cfg.sslCertificate;
+
sslKey = lib.optionalString (cfg.sslCertificateKey != null) cfg.sslCertificateKey;
origin = cfg.hostname;
relayDomains = [ cfg.hostname ];
+1 -1
nixos/modules/services/web-servers/fcgiwrap.nix
···
serviceConfig = {
ExecStart = "${pkgs.fcgiwrap}/sbin/fcgiwrap -c ${builtins.toString cfg.preforkProcesses} ${
-
if (cfg.socketType != "unix") then "-s ${cfg.socketType}:${cfg.socketAddress}" else ""
+
optionalString (cfg.socketType != "unix") "-s ${cfg.socketType}:${cfg.socketAddress}"
}";
} // (if cfg.user != null && cfg.group != null then {
User = cfg.user;
+5 -5
nixos/modules/services/web-servers/lighttpd/default.nix
···
];
maybeModuleString = moduleName:
-
if elem moduleName cfg.enableModules then ''"${moduleName}"'' else "";
+
optionalString (elem moduleName cfg.enableModules) ''"${moduleName}"'';
modulesIncludeString = concatStringsSep ",\n"
(filter (x: x != "") (map maybeModuleString allKnownModules));
···
static-file.exclude-extensions = ( ".fcgi", ".php", ".rb", "~", ".inc" )
index-file.names = ( "index.html" )
-
${if cfg.mod_userdir then ''
+
${optionalString cfg.mod_userdir ''
userdir.path = "public_html"
-
'' else ""}
+
''}
-
${if cfg.mod_status then ''
+
${optionalString cfg.mod_status ''
status.status-url = "/server-status"
status.statistics-url = "/server-statistics"
status.config-url = "/server-config"
-
'' else ""}
+
''}
${cfg.extraConfig}
'';
+2 -2
nixos/modules/services/web-servers/nginx/default.nix
···
listenString = { addr, port, ssl, extraParameters ? [], ... }:
# UDP listener for QUIC transport protocol.
-
(if ssl && vhost.quic then "
+
(optionalString (ssl && vhost.quic) "
listen ${addr}:${toString port} quic "
+ optionalString vhost.default "default_server "
+ optionalString vhost.reuseport "reuseport "
···
let inCompatibleParameters = [ "ssl" "proxy_protocol" "http2" ];
isCompatibleParameter = param: !(any (p: p == param) inCompatibleParameters);
in filter isCompatibleParameter extraParameters))
-
+ ";" else "")
+
+ ";")
+ "
listen ${addr}:${toString port} "
+3 -3
nixos/modules/services/web-servers/tomcat.nix
···
ln -sfn ${tomcat}/conf/$i ${cfg.baseDir}/conf/`basename $i`
done
-
${if cfg.extraConfigFiles != [] then ''
+
${optionalString (cfg.extraConfigFiles != []) ''
for i in ${toString cfg.extraConfigFiles}; do
ln -sfn $i ${cfg.baseDir}/conf/`basename $i`
done
-
'' else ""}
+
''}
# Create a modified catalina.properties file
# Change all references from CATALINA_HOME to CATALINA_BASE and add support for shared libraries
···
# Symlink all the given web applications files or paths into the webapps/ directory
# of this virtual host
-
for i in "${if virtualHost ? webapps then toString virtualHost.webapps else ""}"; do
+
for i in "${optionalString (virtualHost ? webapps) (toString virtualHost.webapps)}"; do
if [ -f $i ]; then
# If the given web application is a file, symlink it into the webapps/ directory
ln -sfn $i ${cfg.baseDir}/virtualhosts/${virtualHost.name}/webapps/`basename $i`
+2 -2
nixos/modules/system/boot/loader/grub/grub.nix
···
then realGrub.override { efiSupport = cfg.efiSupport; }
else null;
-
f = x: if x == null then "" else "" + x;
+
f = x: optionalString (x != null) ("" + x);
grubConfig = args:
let
···
fullName = lib.getName realGrub;
fullVersion = lib.getVersion realGrub;
grubEfi = f grubEfi;
-
grubTargetEfi = if cfg.efiSupport && (cfg.version == 2) then f (grubEfi.grubTarget or "") else "";
+
grubTargetEfi = optionalString (cfg.efiSupport && (cfg.version == 2)) (f (grubEfi.grubTarget or ""));
bootPath = args.path;
storePath = config.boot.loader.grub.storePath;
bootloaderId = if args.efiBootloaderId == null then "${config.system.nixos.distroName}${efiSysMountPoint'}" else args.efiBootloaderId;
+3 -3
nixos/modules/system/boot/loader/systemd-boot/systemd-boot.nix
···
nix = config.nix.package.out;
-
timeout = if config.boot.loader.timeout != null then config.boot.loader.timeout else "";
+
timeout = optionalString (config.boot.loader.timeout != null) config.boot.loader.timeout;
editor = if cfg.editor then "True" else "False";
···
inherit (config.system.nixos) distroName;
-
memtest86 = if cfg.memtest86.enable then pkgs.memtest86-efi else "";
+
memtest86 = optionalString cfg.memtest86.enable pkgs.memtest86-efi;
-
netbootxyz = if cfg.netbootxyz.enable then pkgs.netbootxyz-efi else "";
+
netbootxyz = optionalString cfg.netbootxyz.enable pkgs.netbootxyz-efi;
copyExtraFiles = pkgs.writeShellScript "copy-extra-files" ''
empty_file=$(${pkgs.coreutils}/bin/mktemp)
+1 -2
nixos/modules/system/boot/luksroot.nix
···
copy_bin_and_libs ${pkgs.gnupg}/libexec/scdaemon
${concatMapStringsSep "\n" (x:
-
if x.gpgCard != null then
+
optionalString (x.gpgCard != null)
''
mkdir -p $out/secrets/gpg-keys/${x.device}
cp -a ${x.gpgCard.encryptedPass} $out/secrets/gpg-keys/${x.device}/cryptkey.gpg
cp -a ${x.gpgCard.publicKey} $out/secrets/gpg-keys/${x.device}/pubkey.asc
''
-
else ""
) (attrValues luks.devices)
''}
+1 -1
nixos/modules/tasks/filesystems.nix
···
message = let
fs = head (filter notAutoResizable fileSystems);
in
-
"Mountpoint '${fs.mountPoint}': 'autoResize = true' is not supported for 'fsType = \"${fs.fsType}\"':${if fs.fsType == "auto" then " fsType has to be explicitly set and" else ""} only the ext filesystems and f2fs support it.";
+
"Mountpoint '${fs.mountPoint}': 'autoResize = true' is not supported for 'fsType = \"${fs.fsType}\"':${optionalString (fs.fsType == "auto") " fsType has to be explicitly set and"} only the ext filesystems and f2fs support it.";
}
];
+4 -4
nixos/modules/virtualisation/nixos-containers.nix
···
--setenv HOST_PORT="$HOST_PORT" \
--setenv PATH="$PATH" \
${optionalString cfg.ephemeral "--ephemeral"} \
-
${if cfg.additionalCapabilities != null && cfg.additionalCapabilities != [] then
-
''--capability="${concatStringsSep "," cfg.additionalCapabilities}"'' else ""
+
${optionalString (cfg.additionalCapabilities != null && cfg.additionalCapabilities != [])
+
''--capability="${concatStringsSep "," cfg.additionalCapabilities}"''
} \
-
${if cfg.tmpfs != null && cfg.tmpfs != [] then
-
''--tmpfs=${concatStringsSep " --tmpfs=" cfg.tmpfs}'' else ""
+
${optionalString (cfg.tmpfs != null && cfg.tmpfs != [])
+
''--tmpfs=${concatStringsSep " --tmpfs=" cfg.tmpfs}''
} \
${containerInit cfg} "''${SYSTEM_PATH:-/nix/var/nix/profiles/system}/init"
'';
+2 -2
nixos/modules/virtualisation/qemu-vm.nix
···
mkdir $out
diskImage=$out/disk.img
${qemu}/bin/qemu-img create -f qcow2 $diskImage "120M"
-
${if cfg.useEFIBoot then ''
+
${lib.optionalString cfg.useEFIBoot ''
efiVars=$out/efi-vars.fd
cp ${cfg.efi.variables} $efiVars
chmod 0644 $efiVars
-
'' else ""}
+
''}
'';
buildInputs = [ pkgs.util-linux ];
QEMU_OPTS = "-nographic -serial stdio -monitor none"