----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46003/#review135555 -----------------------------------------------------------
Fix it, then Ship it! src/local/local.cpp (line 351) <https://reviews.apache.org/r/46003/#comment200581> use quotes instead of braces to wrap the work dir. src/local/local.cpp (line 360) <https://reviews.apache.org/r/46003/#comment200582> ditt. use quotes. src/slave/flags.cpp (line 184) <https://reviews.apache.org/r/46003/#comment200583> s/slave/agent/ src/slave/flags.cpp (line 188) <https://reviews.apache.org/r/46003/#comment200584> s/slaves/agents/ src/slave/flags.cpp (line 189) <https://reviews.apache.org/r/46003/#comment200585> s/slave/agent/ - Vinod Kone On May 27, 2016, 11:47 a.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46003/ > ----------------------------------------------------------- > > (Updated May 27, 2016, 11:47 a.m.) > > > Review request for mesos, Michael Park and Vinod Kone. > > > Bugs: MESOS-5064 > https://issues.apache.org/jira/browse/MESOS-5064 > > > Repository: mesos > > > Description > ------- > > The default value for the agent's `--work_dir` flag was removed, the > type of the parameter was changed to `Option<string>`, and code was > added to explicitly set the work directory when necessary. > > > Diffs > ----- > > src/local/local.cpp c0c2766c9317b53a27eb50f6df1eb6ffd556a4cc > src/slave/flags.cpp d3543a019e57658cf60879f40c5f0834a1b8e836 > src/tests/containerizer/mesos_containerizer_tests.cpp > 09742ff21513dc2570684d384b257868dd57a9ce > > Diff: https://reviews.apache.org/r/46003/diff/ > > > Testing > ------- > > `sudo make check` > > > Thanks, > > Greg Mann > >