----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50609/#review144245 -----------------------------------------------------------
src/slave/main.cpp (line 190) <https://reviews.apache.org/r/50609/#comment210243> can you also update master main for consistency? - Vinod Kone On July 29, 2016, 5:16 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50609/ > ----------------------------------------------------------- > > (Updated July 29, 2016, 5:16 p.m.) > > > Review request for mesos, Jie Yu and Pierre Cheynier. > > > Bugs: MESOS-5928 > https://issues.apache.org/jira/browse/MESOS-5928 > > > Repository: mesos > > > Description > ------- > > The recent removal of the agent's default `work_dir` > requires us to check for the presence of the `--version` > flag before we check for errors in flag loading; this > patch makes that change. > > > Diffs > ----- > > src/slave/main.cpp a4d971a00f18af661a8bf2cb4f580d35f7c76d10 > > Diff: https://reviews.apache.org/r/50609/diff/ > > > Testing > ------- > > Ran the mesos agent binary with and without the '--version' flag to verify > correct behavior and return code. > > > Thanks, > > Greg Mann > >
