Revert "fix(builder): properly escape backticks in envvars"

This reverts commit 042ef60.

Author: Matthew Fisher

builder/bin/get-app-values.go

@@ -26,9 +26,7 @@ func main() {
 		os.Exit(1)
 	}
 
-	var retVal string
 	for _, value := range values {
-		retVal = fmt.Sprintf("%s%s", retVal, value)
+		fmt.Println(value)
 	}
-	fmt.Println(retVal)
 }

builder/image/templates/builder

@@ -27,6 +27,14 @@ usage() {
     echo "Usage: $0 <user> <repo> <sha>"
 }
 
+parse-string(){
+    # helper to avoid the single quote escape
+    # occurred in command substitution
+    local args=() idx=0 IFS=' ' c
+    for c; do printf -v args[idx++] '%s ' "$c"; done
+    printf "%s\n" "${args[*]}"
+}
+
 if [ $# -ne $ARGS ]; then
     usage
     exit 1
@@ -105,7 +113,7 @@ if [ ! -f Dockerfile ]; then
     BUILD_OPTS+=' deis/slugbuilder'
 
     # build the application and attach to the process
-    JOB=$(${BUILD_OPTS[@]})
+    JOB=$(eval '$(parse-string "${BUILD_OPTS[@]}")' )
     docker attach $JOB
 
     # copy out the compiled slug

builder/utils.go

@@ -109,7 +109,7 @@ func ParseControllerConfig(bytes []byte) ([]string, error) {
 
 	retVal := []string{}
 	for k, v := range controllerConfig.Values {
-		retVal = append(retVal, fmt.Sprintf(" -e %s='%v'", k, v))
+		retVal = append(retVal, fmt.Sprintf(" -e %s=\"%v\"", k, v))
 	}
 	return retVal, nil
 }

builder/utils_test.go

@@ -163,8 +163,8 @@ func TestParseControllerConfigGood(t *testing.T) {
 		t.Errorf("expected 2, got %d", len(config))
 	}
 
-	if !stringInSlice(config, " -e CAR='star'") {
-		t.Error("expected ' -e CAR='star'' in slice")
+	if !stringInSlice(config, " -e CAR=\"star\"") {
+		t.Error("expected ' -e CAR=\"star\"' in slice")
 	}
 }