----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10077/#review18288 -----------------------------------------------------------
src/tests/utils.hpp <https://reviews.apache.org/r/10077/#comment38477> Why did you need to add this? Can't tests just do os::cwd() ? src/tests/utils.cpp <https://reviews.apache.org/r/10077/#comment38476> not yours, but can you s/cerr/cout - Vinod Kone On March 22, 2013, 7:12 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10077/ > ----------------------------------------------------------- > > (Updated March 22, 2013, 7:12 a.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > The TemporaryDirectoryTest fixture leaks the sandbox directories. > > > Diffs > ----- > > src/tests/utils.hpp 50203579f1ea14ee751c12a373047e9295e4ebb3 > src/tests/utils.cpp 910396d872a8732ed53da353c4225dd81ff10b32 > > Diff: https://reviews.apache.org/r/10077/diff/ > > > Testing > ------- > > make check > verified this was cleaning up > > > Thanks, > > Ben Mahler > >
