Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

Commit

Permalink
Make gradle use travis/docker-thrift/thrift script (#261)
Browse files Browse the repository at this point in the history
Signed-off-by: Yuri Shkuro <[email protected]>
  • Loading branch information
yurishkuro authored Oct 9, 2017
1 parent a5e2a85 commit 4546b8b
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 4 deletions.
2 changes: 0 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,6 @@ cache:
before_install:
- ./travis/prepare-signing.sh $encrypted_677f232983c0_key $encrypted_677f232983c0_iv
- if [ "$CROSSDOCK" == true ]; then bash ./travis/install-crossdock-deps.sh ; fi
- sudo rm -rf /usr/local/bin/thrift
- sudo cp travis/docker-thrift/thrift /usr/local/bin/
script:
- if [ "$TESTS" == true ]; then make test ; else echo 'skipping tests'; fi
- if [ "$COVERAGE" == true ]; then ./gradlew codeCoverageReport ; else echo 'skipping coverage'; fi
Expand Down
2 changes: 1 addition & 1 deletion build.gradle
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import java.util.regex.Pattern

plugins {
id "org.jruyi.thrift" version "0.3.1"
id "org.jruyi.thrift" version "0.4.0"
id "jacoco"
id "com.github.hierynomus.license" version "0.14.0"
id 'com.github.sherter.google-java-format' version '0.3.2'
Expand Down
1 change: 1 addition & 0 deletions jaeger-thrift/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ dependencies {
* `brew install https://raw.githubusercontent.com/Homebrew/homebrew-core/56d8c1eba1e5ac30290dd0c486f4bba37f821e42/Formula/thrift.rb`
*/
compileThrift {
thriftExecutable "${projectDir}/../travis/docker-thrift/thrift"
sourceDir "${projectDir}/../idl/thrift"
outputDir 'src/main/gen-java'
generator 'java', 'private-members'
Expand Down
4 changes: 3 additions & 1 deletion travis/docker-thrift/thrift
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,6 @@ set -e
${THRIFT} $args

# docker command runs as root, but we need normal permissions
sudo chmod -R 0777 ${pwd}/jaeger-thrift/src/main/gen-java
if test ! -r ${pwd}/jaeger-thrift/src/main/gen-java; then
sudo chmod -R 0777 ${pwd}/jaeger-thrift/src/main/gen-java
fi

0 comments on commit 4546b8b

Please sign in to comment.