----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39201/#review102154 -----------------------------------------------------------
Patch looks great! Reviews applied: [39019, 39076, 39091, 39092, 39093, 39096, 39097, 39180, 39181, 39201] All tests passed. - Mesos ReviewBot On Oct. 10, 2015, 4:31 a.m., Alex Clemmer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39201/ > ----------------------------------------------------------- > > (Updated Oct. 10, 2015, 4:31 a.m.) > > > Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph > Wu. > > > Repository: mesos > > > Description > ------- > > The Windows integration work will require us to support `stout/net.hpp`, > but this file includes `stout/os.hpp`, which would be onerous to port. > > To avoid having to port that entire file, we simply roped in what we > needed from `stout/net.hpp`. A consequence of unincluding os.hpp is that > `process/future.hpp` now requires a direct inclusion of > `stout/check.hpp`. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/future.hpp > 9006b8a83d03eab6e67de12a954110029b7d150e > > Diff: https://reviews.apache.org/r/39201/diff/ > > > Testing > ------- > > > Thanks, > > Alex Clemmer > >
