From d6ebbef89de401199f729f2d70239f3e09460164 Mon Sep 17 00:00:00 2001 From: Gabriel Aszalos Date: Fri, 3 Nov 2017 09:40:52 +0100 Subject: [PATCH] runtime: clarify GOROOT return value in documentation The current GOROOT documentation could indicate that changing the environment variable at runtime would affect the return value of GOROOT. This is false as the returned value is the one used for the build. This CL aims to clarify the confusion. Fixes #22302 Change-Id: Ib68c30567ac864f152d2da31f001a98531fc9757 Reviewed-on: https://go-review.googlesource.com/75751 Reviewed-by: Russ Cox --- src/runtime/extern.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/runtime/extern.go b/src/runtime/extern.go index 6e6c674d96..66b9527802 100644 --- a/src/runtime/extern.go +++ b/src/runtime/extern.go @@ -212,8 +212,8 @@ func Callers(skip int, pc []uintptr) int { return callers(skip, pc) } -// GOROOT returns the root of the Go tree. -// It uses the GOROOT environment variable, if set, +// GOROOT returns the root of the Go tree. It uses the +// GOROOT environment variable, if set at process start, // or else the root used during the Go build. func GOROOT() string { s := gogetenv("GOROOT")