> On May 22, 2014, 10:51 a.m., Timothy Chen wrote: > > Can we merge this? This is blocking building for anyone using Mac.
also clang on linux. - Dominic ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21777/#review43736 ----------------------------------------------------------- On May 21, 2014, 11:37 a.m., Dominic Hamon wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21777/ > ----------------------------------------------------------- > > (Updated May 21, 2014, 11:37 a.m.) > > > Review request for mesos and Benjamin Hindman. > > > Repository: mesos-git > > > Description > ------- > > see summary > > Without the explicit conversion some compilers find the conversion from > Unreachable() to Result<T>() ambiguous. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/os.hpp > 94d8b4c7fcf83946e2f9dd0f3e746889599ca174 > > Diff: https://reviews.apache.org/r/21777/diff/ > > > Testing > ------- > > able to build on clang 3.4 > > > Thanks, > > Dominic Hamon > >
