Re: Review Request 47844: Agent: Added support for `slave/main.cpp`.

2016-05-25 Thread Daniel Pravat
> On May 25, 2016, 9:04 p.m., Joris Van Remoortere wrote: > > src/slave/main.cpp, lines 153-162 > > > > > > Can we add a JIRA (and reference it in a comment) to add support for > > this? > > I know a lot of

Re: Review Request 47844: Agent: Added support for `slave/main.cpp`.

2016-05-25 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47844/#review134836 --- Fix it, then Ship it! src/slave/main.cpp (lines 153 - 162)

Re: Review Request 47844: Agent: Added support for `slave/main.cpp`.

2016-05-25 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47844/#review134810 --- Bad patch! Reviews applied: [47844, 47843, 47842, 47442, 47489,

Review Request 47844: Agent: Added support for `slave/main.cpp`.

2016-05-25 Thread Alex Clemmer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47844/ --- Review request for mesos, Daniel Pravat, Artem Harutyunyan, Joris Van