-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40323/
-----------------------------------------------------------

(Updated Nov. 16, 2015, 8:28 a.m.)


Review request for Aurora, Bill Farner and Zameer Manji.


Changes
-------

Fixup BISON_PREFIX to use a unique (case aside) dirent name.

 build-support/thrift/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


Repository: aurora


Description
-------

This improves the isolation of the thrift build by building a local
pinned bison dep. For one, this gets thrift 0.9.1 building on Arch
Linux, a rolling release that is otherwise too modern in its libs to
successfully build thrift 0.9.1.

Additionally, this change knocks out the checksumming TODO and now
checks all downloaded tarballs meet their expected hashes.  The
hashes were generated locally after checking the sha1's where
available.

 build-support/thrift/.gitignore |  7 ++++---
 build-support/thrift/Makefile   | 47 
++++++++++++++++++++++++++++++++++++++---------
 2 files changed, 42 insertions(+), 12 deletions(-)


Diffs (updated)
-----

  build-support/thrift/.gitignore ad6155ebe8671514b5bf5751f250b6cd71c858b7 
  build-support/thrift/Makefile 51f37a959041536004f83a60d038f2116be8c8a9 
  pants.ini 319d38e9a7af8055cac5bbce4a6ae0cbb38dc8d0 

Diff: https://reviews.apache.org/r/40323/diff/


Testing
-------

I can now run `./build-support/jenkins/build.sh` green locally.

I also got a hold of an OSX 10.10.5 box and was able to run
`./build-support/jenkins/build.sh` green on it using this change.


Thanks,

John Sirois

Reply via email to