----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55536/#review161655 -----------------------------------------------------------
Ship it! Ship It! - Zameer Manji On Jan. 14, 2017, 10:19 a.m., John Sirois wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55536/ > ----------------------------------------------------------- > > (Updated Jan. 14, 2017, 10:19 a.m.) > > > Review request for Aurora, Joshua Cohen, Stephan Erb, and Zameer Manji. > > > Bugs: AURORA-1875 > https://issues.apache.org/jira/browse/AURORA-1875 > > > Repository: aurora > > > Description > ------- > > Now the selected thrift is checked both for the proper version and > support of the gen langs Aurora requires. In addition, all thrifts on > the `PATH` are and an existing locally built thrift is always verified > to protect Aurora thrift requirement changes (if we ever add a gen lang > again). > > build-support/thrift/thriftw | 54 > +++++++++++++++++++++++++++++++++++++++----------- > 1 file changed, 42 insertions(+), 12 deletions(-) > > > Diffs > ----- > > build-support/thrift/thriftw 1185298b902b5f1075816d0113ddc34dd4c26a89 > > Diff: https://reviews.apache.org/r/55536/diff/ > > > Testing > ------- > > I have thrift latest (0.10.0) installed as a system package: > ``` > $ ./build-support/thrift/thriftw 0.10.0 --which > /usr/bin/thrift > $ ./build-support/thrift/thriftw 0.10.0 -help 2>&1 | grep Swift > swift (Swift): > $ ./build-support/thrift/thriftw 0.9.1 --which > /home/jsirois/dev/apache/jsirois-aurora/build-support/thrift/thrift-0.9.1/compiler/cpp/thrift > $ ./build-support/thrift/thriftw 0.9.1 -help 2>&1 | grep Swift > $ > ``` > > I also edited the "java (Java)" gen identifier to "java (Java2)" and > observed the script forcing a clean recompile of 0.9.1 with a subsequent > failure to find a compatible thrift. > > > Thanks, > > John Sirois > >