Mailing-List: contact reviews-help@aurora.apache.org; run by ezmlm
Precedence: bulk
Reply-To: reviews@aurora.apache.org
Content-Type: multipart/alternative;
 boundary="===============8886557313356570829=="
MIME-Version: 1.0
Subject: Re: Review Request 55536: Improve `thriftw` robustness.
From: Aurora ReviewBot <wfarner@apache.org>
To: Joshua Cohen <jcohen@apache.org>, Stephan Erb <serb@apache.org>,
 Zameer Manji <zmanji@apache.org>
Cc: Aurora ReviewBot <wfarner@apache.org>, John Sirois <jsirois@apache.org>,
 Aurora <reviews@aurora.apache.org>
Date: Sat, 14 Jan 2017 18:34:52 -0000
Message-ID: <20170114183452.1677.61552@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: Aurora ReviewBot <noreply@reviews.apache.org>
References: <20170114181915.25657.26180@reviews.apache.org>
In-Reply-To: <20170114181915.25657.26180@reviews.apache.org>
Reply-To: Aurora ReviewBot <wfarner@apache.org>
archived-at: Sat, 14 Jan 2017 18:35:01 -0000

--===============8886557313356570829==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


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


Ship it!


Master (b75dfb8) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot retry"

- Aurora ReviewBot


On Jan. 14, 2017, 6:19 p.m., John Sirois wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/55536/
> -----------------------------------------------------------
> 
> (Updated Jan. 14, 2017, 6:19 p.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
> 
>


--===============8886557313356570829==--