diff --git a/lib/strings.nix b/lib/strings.nix index 328f64cf1b61..295d98900e99 100644 --- a/lib/strings.nix +++ b/lib/strings.nix @@ -339,9 +339,10 @@ rec { /* Translate a Nix value into a shell variable declaration, with proper escaping. - Supported value types are strings (mapped to regular variables), lists of strings - (mapped to Bash-style arrays) and attribute sets of strings (mapped to Bash-style - associative arrays). Note that "strings" include string-coercible values like paths. + The value can be a string (mapped to a regular variable), a list of strings + (mapped to a Bash-style array) or an attribute set of strings (mapped to a + Bash-style associative array). Note that "string" includes string-coercible + values like paths or derivations. Strings are translated into POSIX sh-compatible code; lists and attribute sets assume a shell that understands Bash syntax (e.g. Bash or ZSH). @@ -356,7 +357,7 @@ rec { */ toShellVar = name: value: lib.throwIfNot (isValidPosixName name) "toShellVar: ${name} is not a valid shell variable name" ( - if isAttrs value then + if isAttrs value && ! isCoercibleToString value then "declare -A ${name}=(${ concatStringsSep " " (lib.mapAttrsToList (n: v: "[${escapeShellArg n}]=${escapeShellArg v}" diff --git a/lib/tests/misc.nix b/lib/tests/misc.nix index faa7ee547f57..c7010f41394a 100644 --- a/lib/tests/misc.nix +++ b/lib/tests/misc.nix @@ -269,6 +269,15 @@ runTests { strings possibly newlines ''; + drv = { + outPath = "/drv"; + foo = "ignored attribute"; + }; + path = /path; + stringable = { + __toString = _: "hello toString"; + bar = "ignored attribute"; + }; }} ''; expected = '' @@ -277,6 +286,9 @@ runTests { declare -A assoc=(['with some']='strings possibly newlines ') + drv='/drv' + path='/path' + stringable='hello toString' ''; };