----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68644/#review209635 -----------------------------------------------------------
3rdparty/libprocess/src/posix/subprocess.hpp Lines 19 (patched) <https://reviews.apache.org/r/68644/#comment294175> Should this be inside the `__linux__` guard? - James Peach On Oct. 14, 2018, 2:05 p.m., Qian Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68644/ > ----------------------------------------------------------- > > (Updated Oct. 14, 2018, 2:05 p.m.) > > > Review request for mesos, Gilbert Song and James Peach. > > > Bugs: MESOS-9152 and MESOS-9164 > https://issues.apache.org/jira/browse/MESOS-9152 > https://issues.apache.org/jira/browse/MESOS-9164 > > > Repository: mesos > > > Description > ------- > > Closed all file descriptors except `whitelist_fds` in posix/subprocess. > > > Diffs > ----- > > 3rdparty/libprocess/src/posix/subprocess.hpp > 007058b61fdcd4716aa793516c842c3cef8c0a29 > 3rdparty/libprocess/src/subprocess.cpp > c0640de2dc4278b884282dfaad98c49c3b067a5b > > > Diff: https://reviews.apache.org/r/68644/diff/3/ > > > Testing > ------- > > > Thanks, > > Qian Zhang > >
