> On Jan. 21, 2015, 2:17 a.m., Ben Mahler wrote: > > The plan was to split os.hpp into smaller files, and have os.hpp be a full > > include of the os namespace. Can you add rename.hpp to the list of includes > > in os.hpp?
Ah, I figured we were in the process of breaking down `os.hpp` but I didn't realize we wanted `os.hpp` to be the pool of includes! Fixed here: https://reviews.apache.org/r/30108/ Sorry about that. - Michael ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30081/#review68846 ----------------------------------------------------------- On Jan. 20, 2015, 6:35 p.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30081/ > ----------------------------------------------------------- > > (Updated Jan. 20, 2015, 6:35 p.m.) > > > Review request for mesos, Ben Mahler and Jie Yu. > > > Repository: mesos-git > > > Description > ------- > > Will be used for checkpointing in slave. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/Makefile.am > 00a4edd0bcde3f92630fb173aeb4fff8e9139f77 > 3rdparty/libprocess/3rdparty/stout/include/stout/os/rename.hpp PRE-CREATION > > Diff: https://reviews.apache.org/r/30081/diff/ > > > Testing > ------- > > > Thanks, > > Michael Park > >
