Skip to content

Commit

Permalink
Merge pull request #678 from go-kit/test-commit
Browse files Browse the repository at this point in the history
Fix Thrift, if it needs fixing
  • Loading branch information
basvanbeek authored Mar 13, 2018
2 parents 876cd87 + abe6095 commit 52bb1a8
Show file tree
Hide file tree
Showing 4 changed files with 56 additions and 17 deletions.
15 changes: 14 additions & 1 deletion examples/addsvc/thrift/compile.sh
Original file line number Diff line number Diff line change
@@ -1,5 +1,18 @@
#!/usr/bin/env sh

# See also https://thrift.apache.org/tutorial/go
# See also https://thrift.apache.org/tutorial/go.
#
# An old version can be obtained via `brew install thrift`.
# For the latest, here's the annoying dance:
#
# brew install automake bison pkg-config openssl
# ln -s /usr/local/opt/openssl/include/openssl /usr/local/include # if it isn't already
# git clone git@github.com:apache/thrift
# ./bootstrap.sh
# bash
# export PATH=/usr/local/Cellar/bison/*/bin:$PATH
# ./configure ./configure --without-qt4 --without-qt5 --without-c_glib --without-csharp --without-java --without-erlang --without-nodejs --without-lua --without-python --without-perl --without-php --without-php_extension --without-dart --without-ruby --without-haskell --without-rs --without-cl --without-haxe --without-dotnetcore --without-d
# make
# sudo make install

thrift -r --gen "go:package_prefix=github.com/go-kit/kit/examples/addsvc/thrift/gen-go/,thrift_import=github.com/apache/thrift/lib/go/thrift" addsvc.thrift

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion examples/addsvc/thrift/gen-go/addsvc/addsvc-consts.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

25 changes: 13 additions & 12 deletions examples/addsvc/thrift/gen-go/addsvc/addsvc.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 52bb1a8

Please sign in to comment.