Merge pull request #30224 from tmrts/util-exec/fix-dependency

Automatic merge from submit-queue pkg/util/exec: don't specify paths for echo `ex.Command()` already searches the binary in PATH, no need to manually specify it as `/bin/echo`. `pkg/util/exec` tests fail in non-conventional environments due to this (e.g. NixOS).

Merge pull request #30224 from tmrts/util-exec/fix-dependency
c4f48509 · Kubernetes Submit Queue · GitHub · 1bd37f47 · 81fb8e1e · c4f48509
Commit c4f48509 authored Aug 08, 2016 by Kubernetes Submit Queue Committed by GitHub Aug 08, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

exec_test.go pkg/util/exec/exec_test.go +1 -1

No files found.
--- a/pkg/util/exec/exec_test.go
+++ b/pkg/util/exec/exec_test.go
@@ -64,7 +64,7 @@ func TestExecutorNoArgs(t *testing.T) {
 func TestExecutorWithArgs(t *testing.T) {
 	ex := New()
-	cmd := ex.Command("/bin/echo", "stdout")
+	cmd := ex.Command("echo", "stdout")
 	out, err := cmd.CombinedOutput()
 	if err != nil {
 		t.Errorf("expected success, got %+v", err)