From cc9cdca403c98275ec72523bdf4294619639d4a2 Mon Sep 17 00:00:00 2001 From: Rob Pike Date: Wed, 19 Feb 2014 15:33:47 -0800 Subject: cmd/pack: don't look for " in output from go env Windows at least doesn't emit one. Maybe fix Windows build. LGTM=bradfitz R=bradfitz CC=golang-codereviews https://codereview.appspot.com/66120046 --- src/cmd/pack/pack_test.go | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) (limited to 'src/cmd/pack') diff --git a/src/cmd/pack/pack_test.go b/src/cmd/pack/pack_test.go index a073fa452..cab236fa8 100644 --- a/src/cmd/pack/pack_test.go +++ b/src/cmd/pack/pack_test.go @@ -12,7 +12,7 @@ import ( "os" "os/exec" "path/filepath" - "strings" + "regexp" "testing" "time" "unicode/utf8" @@ -193,11 +193,15 @@ func TestHello(t *testing.T) { } out := run("go", "env") - i := strings.Index(out, "GOCHAR=\"") - if i < 0 { + re, err := regexp.Compile(`\s*GOCHAR="?(\w)"?`) + if err != nil { + t.Fatal(err) + } + fields := re.FindStringSubmatch(out) + if fields == nil { t.Fatal("cannot find GOCHAR in 'go env' output:\n", out) } - char := out[i+8 : i+9] + char := fields[1] run("go", "build", "cmd/pack") // writes pack binary to dir run("go", "tool", char+"g", "hello.go") run("./pack", "grc", "hello.a", "hello."+char) -- cgit v1.2.1