No working directory for sh.Exec

[corebreaker]

Hello,

os/exec.Command allow a working directory, but sometimes, we need it in sh.Exec too.

The easy way is to use the exec.Command function, but i don't like it.

I propose to pass a special variable in env parameter, like this:

Exec(map[string]string { "#CWD": "/home/theUser/mydir" }, nil, nil, "my-command")

[natefinch]

I think this is a valid concern and I think there is a way to address it. But the linked PR was not the right solution. I think we can come up with something better and more intuitive.

[corebreaker]

ok, so i'll propose another PR.

[philipsahli]

If have added the PR #246, which should solve this issue. I need to run commands with mage/sh in the working directory, so I pass the variable #CWD with the value from os.Getenv("PWD") to sh.RunWith.

[corebreaker]

Great !

[sluedecke]

I currently use this function as a workaround:

func runCommand(cwd string, command string, args ...string) (ran bool, err error) {
	wd, _ := os.Getwd()
	err = os.Chdir(cwd)
	if err != nil {
		return false, err
	}
	ran, err = sh.Exec(map[string]string{"#CWD": cwd}, os.Stdout, os.Stderr, command, args...)
	os.Chdir(wd) // can return an error, but should never happen.  Smart coders might want to handle it anyways
	return ran, err
}

[waj334]

I currently use this function as a workaround:

func runCommand(cwd string, command string, args ...string) (ran bool, err error) {
	wd, _ := os.Getwd()
	err = os.Chdir(cwd)
	if err != nil {
		return false, err
	}
	ran, err = sh.Exec(map[string]string{"#CWD": cwd}, os.Stdout, os.Stderr, command, args...)
	os.Chdir(wd) // can return an error, but should never happen.  Smart coders might want to handle it anyways
	return ran, err
}

This could cause some commands that run in parallel to fail

[jcchavezs]

I think sh.Exec should have a way to pass the directory. For example, in Coraza we run go test ./... in /my-folder and then go test ./... in /my-folder/my-subfolder (see https://github.com/corazawaf/coraza/pull/422/files#diff-520109d035a196d29d0a86e9c4e6c98e98abf056141b84df68c0d48167b87b25R83) and there is no way to pass a global working directory.
cc @anuraaga as he usually have good ideas on how to overcome API challenges.