shell-variable in egg-compile should quote environment variables

[Mario Domenech Goulart]

Without quoting shell variables, install scripts end up with commands like

mkdir -p ${DESTDIR}'/home/mario/local/chicken-head/lib/chicken/11'

If DESTDIR} happens to contain a space, the command above will not do what it is expected to do.

Double-quoting shell variables on Unix systems is not harmful, as far as I can tell. I have no clue about Windows.

At the moment, shell-variable seems to be only applied to DESTDIR.

The following patch would improve the situation on Unix:

diff --git a/egg-compile.scm b/egg-compile.scm
index 4a72d5d0..e6be5d1d 100644
--- a/egg-compile.scm
+++ b/egg-compile.scm
@@ -1227,7 +1227,7 @@ EOF

 (define (shell-variable var platform)
   (case platform
-    ((unix) (string-append "${" var "}"))
+    ((unix) (string-append "\"${" var "}\""))
     ((windows) (string-append "%" var "%"))))

 ;; NOTE `cmd' must already be quoted for shell

[Vasilij Schneidermann]

You'll need to use double quotes on Windows as well, see ​https://ss64.com/nt/syntax-esc.html for the finer details.

[evhan]

This looks easy enough.

[Vasilij Schneidermann]

Regarding the double-quoting problem, yes, it's an issue on Windows, see ​https://bugs.call-cc.org/ticket/1727#comment:3 for my findings on the topic. A variable must be set correctly and assumed that it doesn't contain any unintended quotes.

[Mario Domenech Goulart]

Fix for unix has been applied (f9a6dd4472bf137bdebd1613e302f1638305b1b9). We still have the issue on Windows.