----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34195/#review84186 -----------------------------------------------------------
Bad patch! Reviews applied: [34193, 34193] Failed command: ./support/apply-review.sh -n -r 34193 Error: 2015-05-18 18:35:17 URL:https://reviews.apache.org/r/34193/diff/raw/ [5544/5544] -> "34193.patch" [1] error: patch failed: 3rdparty/libprocess/3rdparty/stout/include/stout/flags/flags.hpp:42 error: 3rdparty/libprocess/3rdparty/stout/include/stout/flags/flags.hpp: patch does not apply error: patch failed: 3rdparty/libprocess/3rdparty/stout/tests/flags_tests.cpp:483 error: 3rdparty/libprocess/3rdparty/stout/tests/flags_tests.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On May 18, 2015, 5:28 p.m., Marco Massenzio wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34195/ > ----------------------------------------------------------- > > (Updated May 18, 2015, 5:28 p.m.) > > > Review request for mesos, Benjamin Hindman and Joris Van Remoortere. > > > Bugs: MESOS-2711 > https://issues.apache.org/jira/browse/MESOS-2711 > > > Repository: mesos > > > Description > ------- > > Jira: MESOS-2711 > > All the main() methods have been refactored to use the > definition of BaseFlags::help flag and BaseFlags::printUsage(). > > This CL also tries to bring some uniformity to the > use of exit codes: if this is deemed to be worth > making it uniform, we can come up with common > rules and extend the changes here to be compliant. > > This touches a lot of files, but keep scrolling, and you will see a pattern > emerge ;-) > > TODO(marco): we should also abstract away the error checking > > > Diffs > ----- > > src/cli/execute.cpp dbd19e67f56a150f54180ad13e6402842eb68e17 > src/cli/resolve.cpp a99b6094dffc9f7aa44bcf63ad40121e1abb120b > src/examples/load_generator_framework.cpp > be1a3bf5f16bd811cb4039c8f15478183712a426 > src/examples/persistent_volume_framework.cpp > 8a893fcfa3d5d988a88fdeaf0bfc08e0a49b7a65 > src/health-check/main.cpp a4ce742ab8deff1ebd99359112670493fdaeeac3 > src/launcher/executor.cpp de6f1b104a765a8e53934154e78872b03695b24c > src/local/main.cpp a641b9e83862743890597a2981a9419517e7c589 > src/log/tool/benchmark.cpp 01e55115f35d155efbea190b5308b294ba76e7cb > src/log/tool/initialize.cpp ccda7fb1c0f7113f865ec61adee76b2ea6180442 > src/log/tool/read.cpp d14138502f5bc9a725deb83da505765865da017f > src/log/tool/replica.cpp 3985fc7df4f7153ae623589fbdd769ccbae57125 > src/master/main.cpp d5666bc8ee8d7a0f0b8685f76d65dd1f9ac2a280 > src/slave/containerizer/isolators/network/port_mapping.hpp > c72fb47f60f40cda8d84a10497b9133f83cf018e > src/slave/containerizer/isolators/network/port_mapping.cpp > a4abaff30bb4646b1b1edfdbbc243c9e3f6851df > src/slave/main.cpp f762f5b06be74c391cbc336b2da28f8358952ba4 > src/tests/main.cpp e3fff5d60c0468c0d258f2bb301efc1309c071b0 > src/usage/main.cpp 97f55e938dc7678f8331970d8953d09218f70902 > > Diff: https://reviews.apache.org/r/34195/diff/ > > > Testing > ------- > > make check > > **NOTE** this fixes completely the chained changes from 3419{3,4} and makes > all the tests pass. > > > Thanks, > > Marco Massenzio > >
