Skip to content
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

cmd/dist: go tool dist banner: incorrectly reported gobin value #14341

Closed
perillo opened this issue Feb 15, 2016 · 4 comments
Closed

cmd/dist: go tool dist banner: incorrectly reported gobin value #14341

perillo opened this issue Feb 15, 2016 · 4 comments

Comments

@perillo
Copy link

@perillo perillo commented Feb 15, 2016

The go tool dist banner command uses the current value of GOBIN from environment, to report the path where go commands where installed during bootstrap.

It should probably report the gobin value actually used during bootstrap, instead.

@minux
Copy link
Member

@minux minux commented Feb 15, 2016

@perillo
Copy link
Author

@perillo perillo commented Feb 15, 2016

On Mon, Feb 15, 2016 at 9:04 PM, Minux Ma notifications@github.com wrote:

When will those two GOBIN values be different?

When using go from Archlinux:

$go tool dist banner

---
Installed Go for linux/amd64 in /usr/lib/go
Installed commands in /home/manlio/.local/bin

However go core commands (go and gofmt) were installed in
/usr/lib/go/bin, not in /home/manlio/.local/bin.
Probably it is just a misunderstanding about the meaning of go command.

@ianlancetaylor ianlancetaylor added this to the Go1.7 milestone Feb 16, 2016
@ianlancetaylor ianlancetaylor changed the title go tool dist banner: incorrectly reported gobin value cmd/dist: go tool dist banner: incorrectly reported gobin value May 16, 2016
@bradfitz
Copy link
Contributor

@bradfitz bradfitz commented May 20, 2016

Related? #14340 and its https://go-review.googlesource.com/23255 ("build: unset GOBIN during build")

Or not related?

@rsc rsc modified the milestones: Go1.8, Go1.7 May 27, 2016
@rsc
Copy link
Contributor

@rsc rsc commented Oct 18, 2016

Dup of #14340.

@rsc rsc closed this Oct 18, 2016
@golang golang locked and limited conversation to collaborators Oct 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
7 participants
You can’t perform that action at this time.