Re: Review Request 31101: Native Debian packaging for Aurora

2015-06-09 Thread Dan Norris
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31101/#review87189 --- Ship it! I was able to build and deploy debs against master after a

Re: Review Request 31101: Native Debian packaging for Aurora

2015-05-22 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31101/#review85052 --- Ship it! Master (6db13ba) is green with this patch. ./build-suppo

Re: Review Request 31101: Native Debian packaging for Aurora

2015-05-22 Thread Benjamin Staffin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31101/ --- (Updated May 22, 2015, 6:59 p.m.) Review request for Aurora and Jake Farrell.

Re: Review Request 31101: Native Debian packaging for Aurora

2015-05-22 Thread Benjamin Staffin
> On May 22, 2015, 5:47 p.m., Joe Smith wrote: > > debian/rules, line 38 > > > > > > you can also [add the thermos cli > > tool](https://github.com/apache/aurora/blob/master/src/main/python/apache/thermos/cli/bin/BUILD

Re: Review Request 31101: Native Debian packaging for Aurora

2015-05-22 Thread Joe Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31101/#review85036 --- Ship it! LGTM- thanks! debian/rules

Re: Review Request 31101: Native Debian packaging for Aurora

2015-05-21 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31101/#review84744 --- Ship it! Master (998993d) is green with this patch. ./build-suppo

Re: Review Request 31101: Native Debian packaging for Aurora

2015-05-21 Thread Benjamin Staffin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31101/ --- (Updated May 21, 2015, 10:27 a.m.) Review request for Aurora and Jake Farrell.

Re: Review Request 31101: Native Debian packaging for Aurora

2015-05-21 Thread Benjamin Staffin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31101/ --- (Updated May 21, 2015, 10:26 a.m.) Review request for Aurora and Jake Farrell.

Re: Review Request 31101: Native Debian packaging for Aurora

2015-03-30 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31101/#review78245 --- Ship it! Master (3ab8a9e) is green with this patch. ./build-suppo

Re: Review Request 31101: Native Debian packaging for Aurora

2015-03-30 Thread Benjamin Staffin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31101/ --- (Updated March 30, 2015, 12:11 p.m.) Review request for Aurora and Jake Farrell

Re: Review Request 31101: Native Debian packaging for Aurora

2015-03-30 Thread Benjamin Staffin
> On March 2, 2015, 7:19 p.m., Jake Farrell wrote: > > debian/aurora-executor.thermos.default, line 1 > > > > > > needs license header Fixed. > On March 2, 2015, 7:19 p.m., Jake Farrell wrote: > > debian/aurora-execut

Re: Review Request 31101: Native Debian packaging for Aurora

2015-03-30 Thread Benjamin Staffin
> On Feb. 17, 2015, 1:09 p.m., Kevin Sweeney wrote: > > This is great! If this is gonna be mainline and not "contrib", how would > > you feel about adding some tests for this to Jenkins so that reviewbot can > > check this? You should be able to edit build-support/jenkins/build.sh. The > > Jen

Re: Review Request 31101: Native Debian packaging for Aurora

2015-03-30 Thread Benjamin Staffin
> On Feb. 19, 2015, 10:48 a.m., Bill Farner wrote: > > Drive-by piecemeal review - should we create a top-level directory to house > > these? (Presumably a similar dir for RPMs would not be far off.) Perhaps > > `distributions/`? It's appealing from a repo cleanliness perspective, but doing