fix: handle path binaries properly on windows (closes #117, #118)

exec.go

@@ -2,6 +2,7 @@ package main
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"os"
 	"path/filepath"
@@ -17,6 +18,8 @@ import (
 	"mvdan.cc/sh/v3/syntax"
 )
 
+var errNotExec = errors.New("not executable")
+
 func runInput(input, origInput string) {
 	running = true
 	cmdString := aliases.Resolve(input)
@@ -136,8 +139,8 @@ func execCommand(cmd, old string) error {
 			return interp.NewExitStatus(exitcode)
 		}
 
-		err := lookpath(args[0])
+		err, execName := lookpath(args[0])
-		if err == os.ErrPermission {
+		if err == errNotExec {
 			hooks.Em.Emit("command.no-perm", args[0])
 			return interp.NewExitStatus(126)
 		} else if err != nil {
@@ -145,6 +148,8 @@ func execCommand(cmd, old string) error {
 			return interp.NewExitStatus(127)
 		}
 
+		args[0] = execName // windows, thanks
+
 		return interp.DefaultExecHandler(2 * time.Second)(ctx, args)
 	}
 	runner, _ := interp.New(
@@ -156,37 +161,24 @@ func execCommand(cmd, old string) error {
 	return err
 }
 
-// custom lookpath function so we know if a command is found *and* has execute permission
+func lookpath(file string) (error, string) { // custom lookpath function so we know if a command is found *and* is executable
-func lookpath(file string) error {
 	skip := []string{"./", "/", "../", "~/"}
 	for _, s := range skip {
 		if strings.HasPrefix(file, s) {
-			err := findExecutable(file)
+			return findExecutable(file)
-			return err
 		}
 	}
 	for _, dir := range filepath.SplitList(os.Getenv("PATH")) {
 		path := filepath.Join(dir, file)
-		err := findExecutable(path)
+		err, execName := findExecutable(path)
-		if err == os.ErrPermission {
+		if err == errNotExec {
-			return err
+			return err, ""
 		} else if err == nil {
-			return nil
+			return nil, execName
 		}
 	}
 
-	return os.ErrNotExist
+	return os.ErrNotExist, ""
-}
-
-func findExecutable(name string) error {
-	f, err := os.Stat(name)
-	if err != nil {
-		return err
-	}
-	if m := f.Mode(); !m.IsDir() && m & 0111 != 0 {
-		return nil
-	}
-	return os.ErrPermission
 }
 
 func splitInput(input string) ([]string, string) {

execfile_unix.go

@@ -0,0 +1,19 @@
+// +build linux darwin
+
+package main
+
+import (
+	"path/filepath"
+	"os"
+)
+
+func findExecutable(path string) (error, string) {
+	f, err := os.Stat(path)
+	if err != nil {
+		return err, ""
+	}
+	if m := f.Mode(); !m.IsDir() && m & 0111 != 0 {
+		return nil, filepath.Base(path)
+	}
+	return errNotExec, ""
+}

execfile_windows.go

@@ -0,0 +1,26 @@
+// +build windows
+
+package main
+
+import (
+	"path/filepath"
+	"os"
+)
+
+func findExecutable(path string) (error, string) {
+	nameExt := filepath.Ext(path)
+
+	if nameExt == "" {
+		for _, ext := range filepath.SplitList(os.Getenv("PATHEXT")) {
+			_, err := os.Stat(path + ext)
+			if err != nil {
+				return nil, filepath.Base(path + ext)
+			}
+		}
+	} else {
+		_, err := os.Stat(path)
+		return err, ""
+	}
+
+	return errNotExec, ""
+}