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

recipe for Thriftgens failed #11

Open
nishtala opened this issue May 14, 2019 · 8 comments
Open

recipe for Thriftgens failed #11

nishtala opened this issue May 14, 2019 · 8 comments

Comments

@nishtala
Copy link

I'm trying to setup Peloton on a Linux machine.
go version go1.10.4 linux/amd64

The recipe for thirftgens fails here
....
go get ./vendor/go.uber.org/thriftrw
vendor/go.uber.org/thriftrw/main.go:34:2: use of internal package not allowed
vendor/go.uber.org/thriftrw/main.go:35:2: use of internal package not allowed
Makefile:117: recipe for target 'thriftgens' failed
make: *** [thriftgens] Error 1

what am I missing?

@amittbose
Copy link
Contributor

Can you try a later version of Go, e.g. 1.12.4?

@nishtala
Copy link
Author

with Go version 1.12.5, this is the following error

go get ./vendor/go.uber.org/thriftrw
vendor/go.uber.org/thriftrw/main.go:34:2: use of internal package go.uber.org/thriftrw/internal/plugin not allowed
vendor/go.uber.org/thriftrw/main.go:35:2: use of internal package go.uber.org/thriftrw/internal/plugin/builtin/pluginapigen not allowed
Makefile:117: recipe for target 'thriftgens' failed

@onerandomdude
Copy link
Contributor

@nishtala can you make sure the peloton repo is cloned inside the gopath, e.g.

export GOPATH="$HOME/gopath"
export PATH="$PATH:$GOPATH/bin"
mkdir -p "$GOPATH/src/github.com/uber"
git clone https://github.com/uber/peloton.git "$GOPATH/src/github.com/uber/peloton"

@nishtala
Copy link
Author

Yes, it compiles with protoc 3.8.0 and go 1.12.5. But setting it up in the Go path is inconvenient. Two more things (1) is there a publication for peloton? (2) The slack channel is down.

@talaniz
Copy link

talaniz commented Jul 1, 2019

@nishtala could you please provide more information about the use case related to this comment?

But setting it up in the Go path is inconvenient.

What kind of publications did you have in mind? Would a newsletter be of interest? Also, what slack channel are you accessing? It looks like the slack channel is up but I'd like to verify which one you're using.

@nishtala
Copy link
Author

nishtala commented Jul 2, 2019

A publication for this talk: https://www.youtube.com/watch?v=USgbj87Ztlk
A newsletter would be interesting.
This slack channel: https://github.com/uber/peloton#contact

@talaniz
Copy link

talaniz commented Aug 7, 2019

@nishtala I will work on getting a newsletter or communication going. I have tested the slack link and see that it's working on my end now. Can you confirm if it's working for you?

Also, would you be able to help me understand the use case for setting up outside of the gopath? I'd like to document ways that users would need something like this outside of the recommended setup.

@nishtala
Copy link
Author

nishtala commented Aug 8, 2019

Re: Slack. "This invite link is no longer active. Please check with the person who shared it with you to see if there's a new link available."

Re: Go. I don't crisply recall what are the exact changes I did. I can dig into it in two weeks time and get back to you - if that works.

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

No branches or pull requests

4 participants