summary refs log tree commit diff
diff options
context:
space:
mode:
-rw-r--r--pkgs/lib/strings.nix13
1 files changed, 8 insertions, 5 deletions
diff --git a/pkgs/lib/strings.nix b/pkgs/lib/strings.nix
index 263e659f2ec..9c045e9ae53 100644
--- a/pkgs/lib/strings.nix
+++ b/pkgs/lib/strings.nix
@@ -66,11 +66,14 @@ rec {
     else [(substring 0 1 s)] ++ stringToCharacters (substring 1 (builtins.sub l 1) s);
 
     
-  # !!! this function seems broken - it doesn't escape all special
-  # characters, and in any case this should be done in a builder.
-  escapeShellArg = s:
-    let escapeChar = x: if x == "'" then "'\"'\"'" else x;
-    in "'" + concatStrings (map escapeChar (stringToCharacters s)) + "'";
+  # same as vim escape function.
+  # Each character contained in list is prefixed by "\"
+  escape = list : string :
+    lib.concatStrings (map (c: if lib.elem c list then "\\${c}" else c) (stringToCharacters string));
+
+  # still ugly slow. But more correct now
+  # [] for zsh
+  escapeShellArg = lib.escape (stringToCharacters "\\ ';$`()|<>\t*[]");
 
     
   # !!! what is this for?