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

Update README.md install section. #442

Merged
merged 1 commit into from
May 29, 2015

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented May 29, 2015

This commit adds an additional step to the README.md install section to run the go version command and check the version so people that are installing it for the first time and ensure they are running a high enough version.

@jrick
Copy link
Member

jrick commented May 29, 2015

perhaps: "the correct version" -> "a supported version"

@davecgh davecgh force-pushed the update_readme_install branch from 1566984 to e64dcb2 Compare May 29, 2015 20:31
@davecgh
Copy link
Member Author

davecgh commented May 29, 2015

Sounds good.

@davecgh davecgh force-pushed the update_readme_install branch from e64dcb2 to 50008d0 Compare May 29, 2015 20:31
@davecgh
Copy link
Member Author

davecgh commented May 29, 2015

I also added a bit more about checking the GOPATH and GOROOT aren't the same and GOPATH is set to a directory in the home directory.

@jrick
Copy link
Member

jrick commented May 29, 2015

ok

@davecgh davecgh force-pushed the update_readme_install branch 2 times, most recently from 491567a to 3aac3bd Compare May 29, 2015 20:34
This commit adds an additional step to the README.md install section to
run the go version command and check the version so people that are
installing it for the first time and ensure they are running a high
enough version and have GOROOT and GOPATH set correctly.
@conformal-deploy conformal-deploy merged commit 3aac3bd into btcsuite:master May 29, 2015
@davecgh davecgh deleted the update_readme_install branch May 29, 2015 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants