facebookarchive / augmented-traffic-control

Augmented Traffic Control: A tool to simulate network conditions
https://facebook.github.io/augmented-traffic-control
Other
4.33k stars 600 forks source link

Refactor thrift setup. #336

Closed sheenobu closed 6 years ago

sheenobu commented 6 years ago

The old thrift setup used 'atc_thrift' and 'github.com/apache/thrift' as imports, by building an internal GOPATH.

The current thrift setup uses: 'github.com/facebook/augmented-traffic-control/atc-thrift' 'github.com/apache/thrift'.

but this must be changed when using other build tooling. Thrift just does not play well with go's hostname-based imports.

This setup uses the vendor folder to re-enable the naked 'atc_thrift' import. It also uses 'thrift/lib/go/thrift', which allows it to be vendor agnostic.

chantra commented 6 years ago

👍