Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-10 Thread John Sirois
> On Oct. 29, 2015, 4:57 p.m., Aurora ReviewBot wrote: > > Master (bcb4774) is red with this patch. > > ./build-support/jenkins/build.sh > > > > 22:56:59 00:00 [unpack-jars] > > 22:56:59 00:00 [unpack-jars] > > 22:56:59 00:00 [deferred-sources] > > 22:56:59 00:00

Re: Review Request 40149: Create ExecutorSettings creation closer to command line arguments.

2015-11-10 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40149/#review105956 --- Ship it!

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-10 Thread Bill Farner
> On Nov. 10, 2015, 12:18 p.m., John Sirois wrote: > > @ReviewBot retry FYI when you post a diff, the bot will pick it up. The manual intervention should only be necessary for a flaked build. - Bill --- This is an automatically

Re: Review Request 40149: Create ExecutorSettings creation closer to command line arguments.

2015-11-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40149/#review105957 --- Ship it! Master (3ea1b26) is green with this patch.

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-10 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39784/ --- (Updated Nov. 10, 2015, 1:17 p.m.) Review request for Aurora, Joe Smith, Bill

Review Request 40149: Create ExecutorSettings creation closer to command line arguments.

2015-11-10 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40149/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description

Re: Review Request 39784: Upgrade Aurora to pants 0.0.57.

2015-11-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39784/#review105958 --- Master (3ea1b26) is red with this patch.

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-10 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39784/ --- (Updated Nov. 10, 2015, 1:06 p.m.) Review request for Aurora, Joe Smith, Bill

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39784/#review105942 --- Master (3ea1b26) is red with this patch.

Re: Review Request 39784: Upgrade Aurora to pants 0.0.57.

2015-11-10 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39784/ --- (Updated Nov. 10, 2015, 2:50 p.m.) Review request for Aurora, Joe Smith, Bill

Re: Review Request 39784: Upgrade Aurora to pants 0.0.57.

2015-11-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39784/#review105969 --- Master (3ea1b26) is green with this patch.

Re: Review Request 39784: Upgrade Aurora to pants 0.0.57.

2015-11-10 Thread Joe Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39784/#review105973 --- Ship it! Thanks again John! - Joe Smith On Nov. 10, 2015, 1:50

Re: Review Request 39784: Upgrade Aurora to pants 0.0.57.

2015-11-10 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39784/#review105990 --- Ship it! Thanks John! - Zameer Manji On Nov. 10, 2015, 2:58

Re: Review Request 39784: Upgrade Aurora to pants 0.0.57.

2015-11-10 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39784/#review105977 --- Moving the `.auroraversion` file breaks the

Re: Review Request 39784: Upgrade Aurora to pants 0.0.57.

2015-11-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39784/#review105991 --- Master (3ea1b26) is green with this patch.

Re: Review Request 39784: Upgrade Aurora to pants 0.0.57.

2015-11-10 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39784/ --- (Updated Nov. 10, 2015, 3:58 p.m.) Review request for Aurora, Joe Smith, Bill

Re: Review Request 40149: Create ExecutorSettings creation closer to command line arguments.

2015-11-10 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40149/#review105966 --- Ship it!

Re: Review Request 39784: Upgrade Aurora to pants 0.0.57.

2015-11-10 Thread John Sirois
> On Nov. 10, 2015, 3:26 p.m., Zameer Manji wrote: > > Moving the `.auroraversion` file breaks the `build-support/release/release` > > and `build-support/release/release-candidate` scripts. John, I think > > updating those scripts to point to the new location is fine, but I am > > unsure on

Review Request 40161: Clarify the commit process for new contributors.

2015-11-10 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40161/ --- Review request for Aurora, Bill Farner and Zameer Manji. Repository: aurora