Skip to content

Commit dade83a

Browse files
committed
cmd/internal/moddeps: fix false positive when $TMPDIR is symlinked
os.Getwd notes that if the current directory can be reached via multiple paths (due to symbolic links), Getwd may return any one of them. A way to ensure that the desired path is used is to set the PWD environment variable pointing to it. The go generate command has started to update the PWD environment variable as of CL 287152, which was the missing link previously resulting in mkwinsyscall misunderstanding whether it's inside the std lib when symbolic links are involved (issue 44079). Now all that's left is for us to also set the PWD environment variable when invoking the go command in the test, so that it too knows the intended working directory path to use. Fixes #44080. Updates #44079. Updates #43862. Change-Id: I65c9d19d0979f486800b9b328c9b45a1a3180e81 Reviewed-on: https://go-review.googlesource.com/c/go/+/304449 Trust: Dmitri Shuralyov <[email protected]> Reviewed-by: Bryan C. Mills <[email protected]>
1 parent fef5a15 commit dade83a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/cmd/internal/moddeps/moddeps_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -163,8 +163,8 @@ func TestAllDependencies(t *testing.T) {
163163
Env: append(os.Environ(),
164164
// Set GOROOT.
165165
"GOROOT="+gorootCopyDir,
166-
// Explicitly clear PWD and GOROOT_FINAL so that GOROOT=gorootCopyDir is definitely used.
167-
"PWD=",
166+
// Explicitly override PWD and clear GOROOT_FINAL so that GOROOT=gorootCopyDir is definitely used.
167+
"PWD="+filepath.Join(gorootCopyDir, rel),
168168
"GOROOT_FINAL=",
169169
// Add GOROOTcopy/bin and bundleDir to front of PATH.
170170
"PATH="+filepath.Join(gorootCopyDir, "bin")+string(filepath.ListSeparator)+

0 commit comments

Comments
 (0)