----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61892/#review183800 -----------------------------------------------------------
Ship it! Ship It! - Jiang Yan Xu On Aug. 24, 2017, 12:34 p.m., James Peach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61892/ > ----------------------------------------------------------- > > (Updated Aug. 24, 2017, 12:34 p.m.) > > > Review request for mesos, Jie Yu and Jiang Yan Xu. > > > Repository: mesos > > > Description > ------- > > Even when os::rmdir() is given the continueOnError flag, it should still > propagate a failure if it gets one. If we don't do this, the caller has > no way to detect and handle it. > > > Diffs > ----- > > 3rdparty/stout/include/stout/os/posix/rmdir.hpp > 2427bd295f27a1b6b6bcf31b54f445184cdc7e89 > > > Diff: https://reviews.apache.org/r/61892/diff/1/ > > > Testing > ------- > > make check && sudo make check (Fedora 26) > > > Thanks, > > James Peach > >