Skip to content

Update README.md#2

Closed
halgrimur wants to merge 1 commit intoGoogleCloudPlatform:masterfrom
halgrimur:patch-1
Closed

Update README.md#2
halgrimur wants to merge 1 commit intoGoogleCloudPlatform:masterfrom
halgrimur:patch-1

Conversation

@halgrimur
Copy link

the sample won't work unless you specify the gcloud sdk directory in the command line or in pom.xml, UNLESS you installed the gcloud sdk at the default location.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 3, 2015
@tswast
Copy link
Contributor

tswast commented Dec 3, 2015

I swear it used to work without that, but I'm getting the same error "Unkown Google Cloud SDK location" even after following the maven install instructions at https://cloud.google.com/appengine/docs/java/managed-vms/maven

@shun-fan Should we check with the gcloud team? Has there been a maven plugin update?

@tswast
Copy link
Contributor

tswast commented Dec 3, 2015

@halgrimur Thanks for the pull request. I'm going to close this since the issue was fixed by #3.

@tswast tswast closed this Dec 3, 2015
tswast added a commit that referenced this pull request Apr 12, 2016
Remove unnecessary properties.
dzlier-gcp added a commit to dzlier-gcp/getting-started-java that referenced this pull request Oct 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants