Commit b3301865 authored by Michael Hendricks's avatar Michael Hendricks Committed by Brad Fitzpatrick

os: run more of TestExecutable on OpenBSD

On OpenBSD, Executable relies on Args[0].  Removing the forgery on
that OS allows the rest of the test to run.

See #19453

Change-Id: Idf99f86894de5c702893791bc3684f8665f4019d
Reviewed-on: https://go-review.googlesource.com/46398Reviewed-by: default avatarBrad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
parent 1d3a0df4
...@@ -29,12 +29,6 @@ func TestExecutable(t *testing.T) { ...@@ -29,12 +29,6 @@ func TestExecutable(t *testing.T) {
t.Fatalf("filepath.Rel: %v", err) t.Fatalf("filepath.Rel: %v", err)
} }
if runtime.GOOS == "openbsd" {
// The rest of the test doesn't work on OpenBSD,
// which relies on argv[0].
t.Skipf("skipping remainder of test on %s", runtime.GOOS)
}
cmd := &osexec.Cmd{} cmd := &osexec.Cmd{}
// make child start with a relative program path // make child start with a relative program path
cmd.Dir = dir cmd.Dir = dir
...@@ -42,6 +36,10 @@ func TestExecutable(t *testing.T) { ...@@ -42,6 +36,10 @@ func TestExecutable(t *testing.T) {
// forge argv[0] for child, so that we can verify we could correctly // forge argv[0] for child, so that we can verify we could correctly
// get real path of the executable without influenced by argv[0]. // get real path of the executable without influenced by argv[0].
cmd.Args = []string{"-", "-test.run=XXXX"} cmd.Args = []string{"-", "-test.run=XXXX"}
if runtime.GOOS == "openbsd" {
// OpenBSD relies on argv[0]
cmd.Args[0] = fn
}
cmd.Env = append(os.Environ(), fmt.Sprintf("%s=1", executable_EnvVar)) cmd.Env = append(os.Environ(), fmt.Sprintf("%s=1", executable_EnvVar))
out, err := cmd.CombinedOutput() out, err := cmd.CombinedOutput()
if err != nil { if err != nil {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment