From 8012a2e9c96c6c14607388f85079400d2d419c0b Mon Sep 17 00:00:00 2001 From: Russ Cox Date: Mon, 2 Mar 2015 13:49:57 -0500 Subject: [PATCH] cmd/go: avoid creating new empty environment variables Broke some tests that assume $GORACE is unset (because it never is). Those tests are arguably wrong, but this is more robust. Change-Id: Id56daa160c9e7e01f301c1386791e410bbd5deef Reviewed-on: https://go-review.googlesource.com/6480 Reviewed-by: Brad Fitzpatrick --- src/cmd/go/main.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/cmd/go/main.go b/src/cmd/go/main.go index a13d990cfc..5e0ab79df3 100644 --- a/src/cmd/go/main.go +++ b/src/cmd/go/main.go @@ -158,7 +158,9 @@ func main() { // but in practice there might be skew // This makes sure we all agree. for _, env := range mkEnv() { - os.Setenv(env.name, env.value) + if os.Getenv(env.name) != env.value { + os.Setenv(env.name, env.value) + } } for _, cmd := range commands { -- 2.48.1