> On May 7, 2015, 5:06 p.m., Jake Farrell wrote:
> > build-support/release/verify-release-candidate, line 70
> > <https://reviews.apache.org/r/33854/diff/3/?file=950636#file950636line70>
> >
> >     Not crazy about having multiple locations for gradle versions to be 
> > set, do we have to include gradle download here or could we just assume 
> > that gradle is all ready on the system as a build dependency

+1 to the patch other than the gradle download, since we all ready depend on 
vagrant in the e2e test done see why we cant make gradle a similar required 
dependency


- Jake


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


On May 5, 2015, 10:14 p.m., Bill Farner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/33854/
> -----------------------------------------------------------
> 
> (Updated May 5, 2015, 10:14 p.m.)
> 
> 
> Review request for Aurora and Jake Farrell.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> This makes the verification script more complete, automating the basic set of 
> tests that release voters are expected to run.
> 
> 
> Diffs
> -----
> 
>   build-support/release/verify-release-candidate 
> 0d4d6e0c01ebe006056198d25b165b9658156653 
> 
> Diff: https://reviews.apache.org/r/33854/diff/
> 
> 
> Testing
> -------
> 
> TODO(wfarner): Report back results of run against 0.8.0-rc0
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Reply via email to