gexe.Envs Function doesn't work when multiple key/vals are passed as string

[mdzhigarov]

There is an example in the README of how to use the gexe.Envs function:

func main() {
	for _, arch := range []string{"amd64"} {
		for _, opsys := range []string{"darwin", "linux"} {
			gexe.SetVar("arch", arch).SetVar("os", opsys)
			gexe.SetVar("binpath", fmt.Sprintf("build/%s/%s/mybinary", arch, opsys))
			result := gexe.Envs("CGO_ENABLED=0 GOOS=$os GOARCH=$arch").Run("go build -o $binpath .")
			if result != "" {
				fmt.Printf("Build for %s/%s failed: %s\n", arch, opsys, result)
				os.Exit(1)
			}
			fmt.Printf("Build %s/%s: %s OK\n", arch, opsys, echo.Eval("$binpath"))
		}
	}
}

It looks like the parser doesn't work as expected.
gexe.Envs("CGO_ENABLED=0 GOOS=$os GOARCH=$arch") is not parsed correctly and it looks like the the 3 environment variables are not being passed to the process at all. This can easily be proven by inspecting the vars := e.vars.Envs(variables...) after execution with the debugger.

[vladimirvivien]

Hi @mdzhigarov the API's behavior changed in v0.4.1 to fix this. Instead of doing multiple declarations in one string, you add each declarations in separate strings as in:

result := gexe.Envs("CGO_ENABLED=0", "GOOS=$os", "GOARCH=$arch").Run("go build -o $binpath .")

See example here - https://github.com/vladimirvivien/gexe/blob/main/examples/build/main.go

I see the README.md needs updating 😞

[mdzhigarov]

Yep, I realized that but the readme needs a bit of tweaking. Also it'd be good to update the godocs comment inside the functions.go code above Env