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

Fix problem not starting with appengine #1153

Merged
merged 1 commit into from Apr 3, 2017
Merged

Fix problem not starting with appengine #1153

merged 1 commit into from Apr 3, 2017

Conversation

pei0804
Copy link

@pei0804 pei0804 commented Mar 31, 2017

for hashicorp/go-metrics#47

goa uses packages not permitted by GAE of GCP PaaS and can not be used on GAE.
This can be solved by not including syscall containing the code of the package in question in the compilation.
I like Gore, so I would like to use it on GAE. So I requested it.

There is a problem with the dependency package, but if it is solved by goa it will be usable, so please consider.

@raphael
Copy link
Member

raphael commented Apr 3, 2017

Thank you, good catch! Do you think you could backport the fix to the v1 branch as well? (just branch off of v1 and cherry-pick this commit). Thanks again!

@raphael raphael merged commit c62b05f into goadesign:master Apr 3, 2017
@pei0804
Copy link
Author

pei0804 commented Apr 3, 2017

Certainly it seems better to do it.
I'll try.

@pei0804 pei0804 deleted the fix-appengine branch April 3, 2017 02:03
@pei0804 pei0804 restored the fix-appengine branch April 3, 2017 02:03
ikawaha pushed a commit to ikawaha/goa that referenced this pull request Apr 3, 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.

None yet

2 participants