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: misleading warning when compiling with GOROOT_FINAL #5240

Closed
gopherbot opened this issue Apr 8, 2013 · 4 comments
Closed

cmd/dist: misleading warning when compiling with GOROOT_FINAL #5240

gopherbot opened this issue Apr 8, 2013 · 4 comments
Milestone

Comments

@gopherbot
Copy link

@gopherbot gopherbot commented Apr 8, 2013

by kballard:

When running ./all.bash with GODIST_FINAL set and $GOROOT/bin not in $PATH, the final
output of ./all.bash incorrectly claims that I need to add $GOROOT/bin to my PATH. This
is incorrect because I actually need $GODIST_FINAL/bin in path. Specifically, this is
what I'm seeing:

  Installed Go for darwin/amd64 in /Volumes/UserData/Users/kballard/go
  Installed commands in /Volumes/UserData/Users/kballard/go/bin
  *** You need to add /Volumes/UserData/Users/kballard/go/bin to your PATH.
  
  On OS X the debuggers must be installed setgid procmod.
  Read and run ./sudo.bash to install the debuggers.
  
  The binaries expect /Volumes/UserData/Users/kballard/go to be copied or moved to /usr/local/go

The line starting with *** is incorrect, as shown by the final line.


Which operating system are you using?
Mac OS X 10.8.3

Which version are you using?  (run 'go version')
go version devel +d58997478ec6 Mon Apr 08 00:09:35 2013 -0700 darwin/amd64
@ianlancetaylor
Copy link
Contributor

@ianlancetaylor ianlancetaylor commented Apr 8, 2013

Comment 1:

Labels changed: added go1.1.

@adg
Copy link
Contributor

@adg adg commented Apr 9, 2013

Comment 2:

Why is this a 1.1 issue? Doesn't seem very important.

Labels changed: added priority-later, removed priority-triage, go1.1.

Status changed to Accepted.

@rsc
Copy link
Contributor

@rsc rsc commented Jul 30, 2013

Comment 3:

Labels changed: added go1.2.

@ianlancetaylor
Copy link
Contributor

@ianlancetaylor ianlancetaylor commented Sep 5, 2013

Comment 4:

This issue was closed by revision f2380a8.

Status changed to Fixed.

@gopherbot gopherbot added fixed labels Sep 5, 2013
@rsc rsc added this to the Go1.2 milestone Apr 14, 2015
@rsc rsc removed the go1.2 label Apr 14, 2015
@golang golang locked and limited conversation to collaborators Jun 24, 2016
This issue was closed.
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
4 participants
You can’t perform that action at this time.