----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45230/#review125637 -----------------------------------------------------------
Fix it, then Ship it! 3rdparty/libprocess/include/process/subprocess.hpp (line 39) <https://reviews.apache.org/r/45230/#comment188509> brace on new line. ``` enum Setsid { ... }; ``` 3rdparty/libprocess/include/process/subprocess.hpp (line 131) <https://reviews.apache.org/r/45230/#comment188510> `set_sid` and elsehwere. 3rdparty/libprocess/include/process/subprocess.hpp (line 280) <https://reviews.apache.org/r/45230/#comment188511> ticks around `'parent_hooks'` and elsewhere. 3rdparty/libprocess/src/subprocess.cpp (line 315) <https://reviews.apache.org/r/45230/#comment188513> s/slave's/caller's 3rdparty/libprocess/src/subprocess.cpp (line 350) <https://reviews.apache.org/r/45230/#comment188514> no need for `std::` - Joris Van Remoortere On March 28, 2016, 11:42 a.m., Joerg Schad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45230/ > ----------------------------------------------------------- > > (Updated March 28, 2016, 11:42 a.m.) > > > Review request for mesos and Joris Van Remoortere. > > > Repository: mesos > > > Description > ------- > > Executing arbitrary setup functions while creating new processes is > dangerous as all functions called have to be async safe. As setup > functions are used for only very few purposes (setsid, chdir, monitoring > and killing a process (see upcoming review) it makes sense to support > them safely via parameters to subprocess. Note this review by itself > -without the following ones- removing the uses of the old interface will > break the build. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/ssl/gtest.hpp > 2ca705524c8f9bba3c03eef296dc04a353dd236c > 3rdparty/libprocess/include/process/subprocess.hpp > e0c306aa5cf5f393abb73768bbd287c45730f076 > 3rdparty/libprocess/src/subprocess.cpp > b99bad04f7251169df3bfcec5dee459977440997 > > Diff: https://reviews.apache.org/r/45230/diff/ > > > Testing > ------- > > tested entire chain (see https://reviews.apache.org/r/45236/). > > > Thanks, > > Joerg Schad > >
