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

Jump to bottom

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
Labels
FrozenDueToAge
Milestone
Go1.2

Comments

@gopherbot
Copy link

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

Comment 1:

Labels changed: added go1.1.

@adg
Copy link
Contributor

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 commented Jul 30, 2013

Comment 3:

Labels changed: added go1.2.

@ianlancetaylor
Copy link
Contributor

Comment 4:

This issue was closed by revision f2380a8.

Status changed to Fixed.

@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.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
Go1.2
Development

No branches or pull requests
4 participants
@rsc @ianlancetaylor @adg @gopherbot