----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66699/#review201489 -----------------------------------------------------------
Ship it! Caveat: We should file an issue and remove `write.hpp` from both `os.hpp` headers at some point. - Andrew Schwartzmeyer On April 18, 2018, 1:55 p.m., Benno Evers wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66699/ > ----------------------------------------------------------- > > (Updated April 18, 2018, 1:55 p.m.) > > > Review request for mesos. > > > Repository: mesos > > > Description > ------- > > The header 'write.hpp' was included from 'posix/os.hpp', but > not from 'windows/os.hpp', possible leading to subtle build > failures. > > > Diffs > ----- > > 3rdparty/stout/include/stout/windows/os.hpp > 900baf9be4e3089cb43e444b14b155a80bcd1591 > > > Diff: https://reviews.apache.org/r/66699/diff/1/ > > > Testing > ------- > > > Thanks, > > Benno Evers > >
