open-consul/agent/util_other.go

// +build !windows

package agent

import (
	"os"
	"os/exec"
)

// ExecScript returns a command to execute a script through a shell.
func ExecScript(script string) (*exec.Cmd, error) {
	shell := "/bin/sh"
	if other := os.Getenv("SHELL"); other != "" {
		shell = other
	}
	return exec.Command(shell, "-c", script), nil
}
agent: Fix script quoting on windows (#1875) This patch fixes the quoting for executing scripts on windows and splits the platform dependent code. Fixes #1875 2017-08-02 15:01:21 +00:00			`// +build !windows`

			`package agent`

			`import (`
			`"os"`
			`"os/exec"`
			`)`

Clean up subprocess handling and make shell use optional (#3509) * Clean up handling of subprocesses and make using a shell optional * Update docs for subprocess changes * Fix tests for new subprocess behavior * More cleanup of subprocesses * Minor adjustments and cleanup for subprocess logic * Makes the watch handler reload test use the new path. * Adds check tests for new args path, and updates existing tests to use new path. * Adds support for script args in Docker checks. * Fixes the sanitize unit test. * Adds panic for unknown watch type, and reverts back to Run(). * Adds shell option back to consul lock command. * Adds shell option back to consul exec command. * Adds shell back into consul watch command. * Refactors signal forwarding and makes Windows-friendly. * Adds a clarifying comment. * Changes error wording to a warning. * Scopes signals to interrupt and kill. This avoids us trying to send SIGCHILD to the dead process. * Adds an error for shell=false for consul exec. * Adds notes about the deprecated script and handler fields. * De-nests an if statement. 2017-10-04 23:48:00 +00:00			`// ExecScript returns a command to execute a script through a shell.`
agent: Fix script quoting on windows (#1875) This patch fixes the quoting for executing scripts on windows and splits the platform dependent code. Fixes #1875 2017-08-02 15:01:21 +00:00			`func ExecScript(script string) (*exec.Cmd, error) {`
			`shell := "/bin/sh"`
			`if other := os.Getenv("SHELL"); other != "" {`
			`shell = other`
			`}`
			`return exec.Command(shell, "-c", script), nil`
			`}`