----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45230/#review125677 -----------------------------------------------------------
3rdparty/libprocess/include/process/subprocess.hpp (line 301) <https://reviews.apache.org/r/45230/#comment188545> This one breaks build even after all of your 7 patches are committed, because port_mapping.cpp passes flags but no setsid to subprocess(), this is not allowed by C++. 3rdparty/libprocess/include/process/subprocess.hpp (line 342) <https://reviews.apache.org/r/45230/#comment188546> Ditto - Cong Wang On March 28, 2016, 4:51 p.m., Joerg Schad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45230/ > ----------------------------------------------------------- > > (Updated March 28, 2016, 4:51 p.m.) > > > Review request for mesos and Joris Van Remoortere. > > > Bugs: MESOS-5049 > https://issues.apache.org/jira/browse/MESOS-5049 > > > 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 > >