Re: Review Request 46107: Add Appc runtime spec for command, working directory and environment.

2016-06-21 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46107/#review138949 --- Bad patch! Reviews applied: [46107] Failed command: ./support/ap

Re: Review Request 46107: Add Appc runtime spec for command, working directory and environment.

2016-06-21 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46107/ --- (Updated June 21, 2016, 6:10 p.m.) Review request for mesos, Gilbert Song and J

Re: Review Request 46107: Add Appc runtime spec for command, working directory and environment.

2016-06-21 Thread Srinivas Brahmaroutu
> On June 21, 2016, 4:35 p.m., Gilbert Song wrote: > > Seems like this patch is included in https://reviews.apache.org/r/46498/ > > > > Could you discard it if it is not reviewable? Thanks! If this patch looks good, please merge it and I will rebase the other. - Srinivas ---

Re: Review Request 46107: Add Appc runtime spec for command, working directory and environment.

2016-06-21 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46107/#review138867 --- Seems like this patch is included in https://reviews.apache.org/r/

Re: Review Request 46107: Add Appc runtime spec for command, working directory and environment.

2016-06-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46107/#review136011 --- Patch looks great! Reviews applied: [46107] Passed command: expo

Re: Review Request 46107: Add Appc runtime spec for command, working directory and environment.

2016-06-02 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46107/ --- (Updated June 2, 2016, 10:14 p.m.) Review request for mesos, Gilbert Song and J

Re: Review Request 46107: Add Appc runtime spec for command, working directory and environment.

2016-05-13 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46107/#review133232 --- Patch looks great! Reviews applied: [46107] Passed command: expo

Re: Review Request 46107: Add Appc runtime spec for command, working directory and environment.

2016-05-13 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46107/ --- (Updated May 13, 2016, 7:52 p.m.) Review request for mesos, Gilbert Song and Ji

Re: Review Request 46107: Add Appc runtime spec for command, working directory and environment.

2016-05-03 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46107/#review131527 --- include/mesos/appc/spec.proto (line 71)

Re: Review Request 46107: Add Appc runtime spec for command, working directory and environment.

2016-04-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46107/#review128585 --- Patch looks great! Reviews applied: [46107] Passed command: expo

Review Request 46107: Add Appc runtime spec for command, working directory and environment.

2016-04-12 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46107/ --- Review request for mesos. Repository: mesos Description --- Add Appc run