Set absolute bash path

and slightly revert assertion in docs test to ensure flag ordering
urfave · May 23, 2022 · 8f47e6d · 8f47e6d
1 parent 5edc1b9
commit 8f47e6d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/docs/v2/manual.md b/docs/v2/manual.md
@@ -363,7 +363,7 @@ For example this:
 
 <!-- {
   "args": ["&#45;&#45;help"],
-  "output": ".*Language for the greeting.*"
+  "output": ".*Load configuration from FILE\n.*\n.*Language for the greeting.*"
 } -->
 ``` go
 package main

diff --git a/internal/build/build.go b/internal/build/build.go
@@ -182,6 +182,13 @@ func testCleanup(packages []string) error {
 func GfmrunActionFunc(cCtx *cli.Context) error {
 	top := cCtx.Path("top")
 
+	bash, err := exec.LookPath("bash")
+	if err != nil {
+		return err
+	}
+
+	os.Setenv("SHELL", bash)
+
 	tmpDir, err := os.MkdirTemp("", "urfave-cli*")
 	if err != nil {
 		return err
@@ -203,7 +210,6 @@ func GfmrunActionFunc(cCtx *cli.Context) error {
 	}
 
 	os.Setenv("TMPDIR", tmpDir)
-	os.Setenv("SHELL", "bash")
 
 	if err := os.Chdir(wd); err != nil {
 		return err