]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/go: fix TestNoCache on Plan 9
authorDavid du Colombier <0intro@gmail.com>
Wed, 31 Jan 2018 21:56:39 +0000 (22:56 +0100)
committerDavid du Colombier <0intro@gmail.com>
Wed, 31 Jan 2018 22:12:10 +0000 (22:12 +0000)
CL 91097 added TestNoCache. However, this
test is failing on Plan 9 because the HOME
environment variable doesn't contain the
home directory where the Go cache is located.

This change fixes the TestNoCache test
by using the home environment variable
instead of HOME on Plan 9.

Fixes #23644.

Change-Id: Icfb7a7a4c2852f159c93032b4081411628a2787f
Reviewed-on: https://go-review.googlesource.com/91216
Run-TryBot: David du Colombier <0intro@gmail.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/cmd/go/go_test.go

index b2053f553c7c7ec08067e42dd9a7bf614d15b501..ac47adb8e7dcf5ca423962e7cd52ce4bd132052a 100644 (file)
@@ -5387,7 +5387,11 @@ func TestNoCache(t *testing.T) {
        tg.parallel()
        tg.tempFile("triv.go", `package main; func main() {}`)
        tg.must(os.MkdirAll(tg.path("unwritable"), 0555))
-       tg.setenv("HOME", tg.path(filepath.Join("unwritable", "home")))
+       home := "HOME"
+       if runtime.GOOS == "plan9" {
+               home = "home"
+       }
+       tg.setenv(home, tg.path(filepath.Join("unwritable", "home")))
        tg.unsetenv("GOCACHE")
        tg.run("build", "-o", tg.path("triv"), tg.path("triv.go"))
        tg.grepStderr("disabling cache", "did not disable cache")