From 0cf7e54f2fe033a1e596d9aaed0e728cda1fc5e7 Mon Sep 17 00:00:00 2001 From: Ron Minnich Date: Tue, 29 Aug 2017 09:59:27 -0700 Subject: [PATCH] os: don't assume /bin/pwd in test, find it in $PATH MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit There are several distros now that no longer have /bin. Instead of assuming /bin/pwd, we will look for it in $PATH. Fixes #21684. Change-Id: I61478326500edeadc3c26803990550dad00c7971 Signed-off-by: Ron Minnich Reviewed-on: https://go-review.googlesource.com/60010 Reviewed-by: Daniel Martí Run-TryBot: Daniel Martí TryBot-Result: Gobot Gobot --- src/os/os_test.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/os/os_test.go b/src/os/os_test.go index dbe4ff8830..c807786310 100644 --- a/src/os/os_test.go +++ b/src/os/os_test.go @@ -1014,9 +1014,14 @@ func TestStartProcess(t *testing.T) { dir = Getenv("SystemRoot") args = []string{"/c", "cd"} default: - cmd = "/bin/pwd" + var err error + cmd, err = osexec.LookPath("pwd") + if err != nil { + t.Fatalf("Can't find pwd: %v", err) + } dir = "/" args = []string{} + t.Logf("Testing with %v", cmd) } cmddir, cmdbase := filepath.Split(cmd) args = append([]string{cmdbase}, args...)