----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41429/#review113462 -----------------------------------------------------------
Ship it! src/tests/environment.cpp (line 129) <https://reviews.apache.org/r/41429/#comment174184> Include "stout/result.hpp" please. src/tests/environment.cpp (line 132) <https://reviews.apache.org/r/41429/#comment174183> Include "stout/path.hpp" please. src/tests/environment.cpp (line 135) <https://reviews.apache.org/r/41429/#comment174185> Include "stout/none.hpp" please. src/tests/environment.cpp (line 166) <https://reviews.apache.org/r/41429/#comment174182> Include "stout/option.hpp" please. - Till Toenshoff On Jan. 7, 2016, 8 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41429/ > ----------------------------------------------------------- > > (Updated Jan. 7, 2016, 8 p.m.) > > > Review request for mesos, Bernd Mathiske, Ben Mahler, Till Toenshoff, and > Timothy Chen. > > > Repository: mesos > > > Description > ------- > > Cleaned up the CfsFilter and clarified its logging message. Many thanks to > @bmahler for noticing these issues! > > > Diffs > ----- > > src/tests/environment.cpp 36f0ad0b739559e5c883d72585731944da4283cf > > Diff: https://reviews.apache.org/r/41429/diff/ > > > Testing > ------- > > `sudo make check` on both Debian 8.2 and Ubuntu 14.04. The "CFS_" tests run > correctly on Ubuntu, while the appropriate error message is displayed on > Debian and the "CFS_" tests are filtered out. > > > Thanks, > > Greg Mann > >
