diff options
author | Bryan C. Mills <bcmills@google.com> | 2020-03-25 22:17:04 -0400 |
---|---|---|
committer | Bryan C. Mills <bcmills@google.com> | 2020-03-26 19:07:15 +0000 |
commit | 437bd90f13c36cd57298a950caae6601cd60550c (patch) | |
tree | 225597d57b085acd82eec68e347e5338245094dd /src/cmd/go/internal/test/test.go | |
parent | bfb1342a40216cba0ff5ae3a1b102823b7603068 (diff) | |
download | go-437bd90f13c36cd57298a950caae6601cd60550c.tar.gz go-437bd90f13c36cd57298a950caae6601cd60550c.zip |
cmd/go/internal/base: rename EnvForDir to AppendPWD
EnvForDir does not immediately evoke “append”, and thus may not prompt
the reader to consider the possibility of aliasing bugs (as in
issue #38077). To make this behavior more obvious at the call site, rename
cmd/go/internal/base.EnvForDir to AppendPWD and swap the order of
arguments to a conventional “append” function (similar to those in the
strconv package).
For #38077
Change-Id: I16f09aa0fa8a269d51f0511eb402a44e2759eb94
Reviewed-on: https://go-review.googlesource.com/c/go/+/225578
Reviewed-by: Jay Conrod <jayconrod@google.com>
Reviewed-by: Michael Matloob <matloob@golang.org>
Diffstat (limited to 'src/cmd/go/internal/test/test.go')
-rw-r--r-- | src/cmd/go/internal/test/test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/cmd/go/internal/test/test.go b/src/cmd/go/internal/test/test.go index ec7bde16fe..48a873e46b 100644 --- a/src/cmd/go/internal/test/test.go +++ b/src/cmd/go/internal/test/test.go @@ -1157,7 +1157,7 @@ func (c *runCache) builderRunTest(b *work.Builder, a *work.Action) error { cmd := exec.Command(args[0], args[1:]...) cmd.Dir = a.Package.Dir - cmd.Env = base.EnvForDir(cmd.Dir, cfg.OrigEnv[:len(cfg.OrigEnv):len(cfg.OrigEnv)]) + cmd.Env = base.AppendPWD(cfg.OrigEnv[:len(cfg.OrigEnv):len(cfg.OrigEnv)], cmd.Dir) cmd.Stdout = stdout cmd.Stderr = stdout |