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

Updating go vet location in install-dev-tools target, as it has moved. #322

Merged
merged 1 commit into from
Jan 13, 2015
Merged

Conversation

robdaemon
Copy link

Fixes this error:

can't load package: the code.google.com/p/go.tools/cmd/vet command has
moved; use golang.org/x/tools/cmd/vet instead.

Fixes this error:

can't load package: the code.google.com/p/go.tools/cmd/vet command has
moved; use golang.org/x/tools/cmd/vet instead.
@cfdreddbot
Copy link

Hey robdaemon!

Thanks for submitting this pull request!

All pull request authors must have a Contributor License Agreement (CLA) on-file with us. Please sign the appropriate CLA (individual or corporate).

When sending signed CLA please provide your github username in case of individual CLA or the list of github usernames that can make pull requests on behalf of your organization.

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/85570392.

@robdaemon
Copy link
Author

Corporate CLA for HP is already signed and on file, I'll get my individual one signed and emailed in tonight.

@simonleung8
Copy link
Contributor

Thanks for the PR, merged.

@robdaemon
Copy link
Author

@simonleung8 - this looks like it was closed without merging the change?

@simonleung8 simonleung8 reopened this Jan 13, 2015
simonleung8 pushed a commit that referenced this pull request Jan 13, 2015
Updating go vet location in install-dev-tools target, as it has moved.
@simonleung8 simonleung8 merged commit 658e3e1 into cloudfoundry:master Jan 13, 2015
@simonleung8
Copy link
Contributor

You are right, my mistake, it is really merged this time.
thanks for the look out.

@robdaemon robdaemon deleted the fix_go_vet branch January 13, 2015 02:31
@robdaemon
Copy link
Author

I've done that more than once. Thanks!

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.

4 participants