-
Notifications
You must be signed in to change notification settings - Fork 17.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: all.bash fails when GOBIN is set via 'go env -w' #37390
Comments
Perhaps related to #32674? CC @jayconrod @bcmills |
Perhaps the misc tests should be better isolated from the user environment in general? I've also seen issues with |
I don't understand how these tests are passing in the first place, if they're hard-coding |
Oh, that's how: the test is changing to a subdirectory of its own Probably when we call |
Oh, we do that too! go/misc/cgo/testshared/shared_test.go Lines 156 to 158 in d243408
I also can't reproduce the problem by setting I bet the |
Bingo:
|
Change https://golang.org/cl/220652 mentions this issue: |
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes, it reproduces under go1.13 [although it does not leave any test binaries in GOBIN]:
all.bash
OutputWhat operating system and processor architecture are you using (
go env
)?I'm using a non-standard
GOBIN
:go env
OutputWhat did you do?
I run
all.bash
under tip.What did you expect to see?
All tests pass and no extra left over binaries in GOBIN.
What did you see instead?
all.bash
returnedgo tool dist: FAILED
:all.bash
OutputAnd it failed to clean up some test binaries in
GOBIN
:ls -ltr `go env GOBIN` | tail -n6
OutputI'm mentioning this because this seems to be the binaries that
all.bash
failed to found (according to some of the error messages).The text was updated successfully, but these errors were encountered: