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 for installation #2

Closed
wants to merge 1 commit into from
Closed

Update README.md for installation #2

wants to merge 1 commit into from

Conversation

mhemmings
Copy link

go get will get all dependancies, so there is no need for a second step

`go get` will get all dependancies, so there is no need for a second step
dlsniper added a commit to dlsniper/aws-xray-sdk-go that referenced this pull request Sep 4, 2017
@dlsniper
Copy link

dlsniper commented Sep 4, 2017

The problem is that there are no sources in the root of the package github.com/aws/aws-xray-sdk-go so go get will not follow the packages it contains automatically. To do so, either see my commit that references this PR or add go get -u github.com/aws/aws-xray-sdk-go/... to the instructions.

@mhemmings
Copy link
Author

Ah OK I see, that's an awkward one. But I agree dfb09e1 seems to be the suitable way round it.

@mhemmings mhemmings closed this Sep 4, 2017
@luluzhao luluzhao mentioned this pull request Sep 6, 2017
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.

2 participants