Merge pull request #253 from lets-cli/fix-bash-resolving-error

do not set SHELL to avoid bash: file not found error. set LETS_SHELL …

.gitignore

@@ -15,6 +15,7 @@
 # vendor/
 .idea
 .vscode
+!.vscode/launch.json
 .history
 dist
 
@@ -26,3 +27,6 @@ _lets
 coverage.out
 node_modules
 TODO
+
+.DS_Store
+__debug_bin*

.vscode/launch.json

@@ -0,0 +1,23 @@
+{
+    // Use IntelliSense to learn about possible attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": [
+        {
+            "name": "Launch Package",
+            "type": "go",
+            "request": "launch",
+            "mode": "auto",
+            "program": "${fileDirname}"
+        },
+        {
+            "name": "Run",
+            "type": "go",
+            "request": "launch",
+            "mode": "auto",
+            "program": "${workspaceRoot}",
+            "args": []
+        }
+    ]
+}

docs/docs/changelog.md

@@ -5,6 +5,10 @@ title: Changelog
 
 ## [Unreleased](https://github.com/lets-cli/lets/releases/tag/v0.0.X)
 
+## [0.0.53](https://github.com/lets-cli/lets/releases/tag/v0.0.53)
+
+* `[Fixed]` change `SHELL` env to `LETS_SHELL` because setting system variable `SHELL` to just `bash` without full path to binary cases and error in some cases.
+
 ## [0.0.52](https://github.com/lets-cli/lets/releases/tag/v0.0.52)
 
 * `[Dependency]` update and pin goreleaser

docs/docs/env.md

@@ -19,7 +19,7 @@ title: Environment
 * `LETS_COMMAND_WORK_DIR` - absolute path to `work_dir` specified in command.
 * `LETS_CONFIG` - absolute path to lets config file.
 * `LETS_CONFIG_DIR` - absolute path to lets config file firectory.
-* `SHELL` - shell from config or command.
+* `LETS_SHELL` - shell from config or command.
 * `LETSOPT_<>` - options parsed from command `options` (docopt string). E.g `lets run --env=prod --reload` will be `LETSOPT_ENV=prod` and `LETSOPT_RELOAD=true`
 * `LETSCLI_<>` - options which values is a options usage. E.g `lets run --env=prod --reload` will be `LETSCLI_ENV=--env=prod` and `LETSCLI_RELOAD=--reload`
 

executor/executor.go

@@ -208,7 +208,7 @@ func (e *Executor) setupEnv(osCmd *exec.Cmd, command *config.Command, shell stri
 		"LETS_COMMAND_WORK_DIR": osCmd.Dir,
 		"LETS_CONFIG":           filepath.Base(e.cfg.FilePath),
 		"LETS_CONFIG_DIR":       filepath.Dir(e.cfg.FilePath),
-		"SHELL":                 shell,
+		"LETS_SHELL":                 shell,
 	}
 
 	checksumEnvMap := getChecksumEnvMap(command.ChecksumMap)

lets.yaml

@@ -114,4 +114,4 @@ commands:
 
   run-docs:
     work_dir: docs
-    cmd: npm start
+    cmd: npm start

tests/command_shell/lets.yaml

@@ -3,4 +3,4 @@ shell: bash
 commands:
   show-shell:
     shell: /bin/sh
-    cmd: echo "$SHELL"
+    cmd: echo "$LETS_SHELL"