> On Sept. 8, 2015, 11:12 p.m., Artem Harutyunyan wrote: > > This file has not been commited, so I suggest to drop this review and make > > this change in the original review that introduces the affected line. > > Alex Clemmer wrote: > It actually has been committed. haosdent broke the build, that's why it's > here. > > haosdent huang wrote: > Yes, my fix for this is https://reviews.apache.org/r/37502/
No worries haosdent, it will just take me some time to apply your reviews to test them on the platforms that are really important. :) I'll give it a shot before I go to bed tonight. - Alex ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37370/#review98110 ----------------------------------------------------------- On Sept. 10, 2015, 8:53 p.m., Alex Clemmer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37370/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2015, 8:53 p.m.) > > > Review request for mesos, Benjamin Hindman, haosdent huang, Artem > Harutyunyan, Joris Van Remoortere, and Joseph Wu. > > > Repository: mesos > > > Description > ------- > > Right now if you try to compile GMock a non-Windows platform using the > CMake build system, it will error out. The problem seems to be the CXX11 > flag. > > This commit will solve this problem by removing that flag. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/CMakeLists.txt > 997cc0d0e316e316136d4746e50e9e292a82b36b > > Diff: https://reviews.apache.org/r/37370/diff/ > > > Testing > ------- > > > Thanks, > > Alex Clemmer > >
