Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Go Caveats #31288

Closed
wants to merge 1 commit into from
Closed

Go Caveats #31288

wants to merge 1 commit into from

Conversation

DomT4
Copy link
Member

@DomT4 DomT4 commented Aug 1, 2014

I don’t particularly love that this isn’t the cleanest set of warnings in the world, but the combination of Homebrew, zsh & Go seems to really screw up the setting of the GOPATH & the $PATH as recommended. (The problem seems to replicate across any non-default shell). I’ve chucked in a few lines consequently about how to handle non-default shells.

I’ll respect if this somewhat cumbersome explanation is rejected, but hopefully it’s useful to include.

I don’t particularly love that this isn’t the cleanest set of warnings
in the world, but the combination of Homebrew, zsh & Go seems to really
screw up the setting of the GOPATH & the $PATH as recommended. I’ve
chucked in a few lines consequently about how to handle non-default
shells.

I’ll respect if this somewhat cumbersome explanation is rejected, but
hopefully it’s useful to include.
@@ -78,9 +78,16 @@ def caveats; <<-EOS.undent
To get `go vet` and `go doc` run:
go get code.google.com/p/go.tools/cmd/godoc
go get code.google.com/p/go.tools/cmd/vet
Note to use go get you will need 'Mercurial' installed.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that not just for mercurial repos?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems to be for any 'Go' project hosted on code.google.com, and so far I'm struggling to find 'Go' projects that don't tap at least one of the many go resources hosted on code.google.com.

I'm willing to remove the line though. Chances are 90%+ of people using Google Go are likely to have Mercurial installed already and so it may well be redundant.

@MikeMcQuaid
Copy link
Member

Giving this is a miss but appreciate the attempt. Sorry to close!

@MikeMcQuaid MikeMcQuaid closed this Aug 2, 2014
@DomT4 DomT4 deleted the gofixes branch August 2, 2014 17:03
@DomT4
Copy link
Member Author

DomT4 commented Aug 2, 2014

No problem! Thanks for talking it through & explaining it to me. Appreciated.

@Homebrew Homebrew locked and limited conversation to collaborators Feb 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants