Re: Review Request 34984: Added help for files
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34984/#review86849 --- Ship it! I will make the change for you below. Thanks Aditi! src/files/files.cpp https://reviews.apache.org/r/34984/#comment138950 Should be: ``` using process::DESCRIPTION; using process::HELP; using process::TLDR; using process::USAGE; using process::wait; // Necessary on some OS's to disambiguate. ``` Underneath 'using namespace process;' :) src/files/files.cpp https://reviews.apache.org/r/34984/#comment138947 s/Path/path/ - Niklas Nielsen On June 4, 2015, 2:40 a.m., Aditi Dixit wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34984/ --- (Updated June 4, 2015, 2:40 a.m.) Review request for mesos, Niklas Nielsen and Vinod Kone. Bugs: MESOS-2277 https://issues.apache.org/jira/browse/MESOS-2277 Repository: mesos Description --- Added help for files Diffs - src/files/files.cpp ce02411c5e579d7551b4325ec141fd89e4ee7255 Diff: https://reviews.apache.org/r/34984/diff/ Testing --- make check Thanks, Aditi Dixit
Re: Review Request 34984: Added help for files
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34984/ --- (Updated June 4, 2015, 9:40 a.m.) Review request for mesos, Niklas Nielsen and Vinod Kone. Changes --- Fixed issues Bugs: MESOS-2277 https://issues.apache.org/jira/browse/MESOS-2277 Repository: mesos Description --- Added help for files Diffs (updated) - src/files/files.cpp ce02411c5e579d7551b4325ec141fd89e4ee7255 Diff: https://reviews.apache.org/r/34984/diff/ Testing --- make check Thanks, Aditi Dixit
Review Request 34984: Added help for files
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34984/ --- Review request for mesos, Niklas Nielsen and Vinod Kone. Bugs: MESOS-2277 https://issues.apache.org/jira/browse/MESOS-2277 Repository: mesos Description --- Added help for files Diffs - src/files/files.cpp ce02411c5e579d7551b4325ec141fd89e4ee7255 Diff: https://reviews.apache.org/r/34984/diff/ Testing --- make check Thanks, Aditi Dixit
Re: Review Request 34984: Added help for files
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34984/#review86374 --- Patch looks great! Reviews applied: [34984] All tests passed. - Mesos ReviewBot On June 3, 2015, 6:11 a.m., Aditi Dixit wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34984/ --- (Updated June 3, 2015, 6:11 a.m.) Review request for mesos, Niklas Nielsen and Vinod Kone. Bugs: MESOS-2277 https://issues.apache.org/jira/browse/MESOS-2277 Repository: mesos Description --- Added help for files Diffs - src/files/files.cpp ce02411c5e579d7551b4325ec141fd89e4ee7255 Diff: https://reviews.apache.org/r/34984/diff/ Testing --- make check Thanks, Aditi Dixit