----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55040/#review160221 -----------------------------------------------------------
Patch looks great! Reviews applied: [55022, 55023, 55024, 55025, 55026, 55028, 55029, 55030, 55037, 55038, 55039, 55040] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On Dec. 26, 2016, 9:54 a.m., Alex Clemmer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55040/ > ----------------------------------------------------------- > > (Updated Dec. 26, 2016, 9:54 a.m.) > > > Review request for mesos, Andrew Schwartzmeyer, Daniel Pravat, John Kordich, > and Joseph Wu. > > > Bugs: MESOS-5455 > https://issues.apache.org/jira/browse/MESOS-5455 > > > Repository: mesos > > > Description > ------- > > Currently, per MESOS-5455, CMake does not implement certain build flags > as the autotools solution does. For example, `BUILD_DATE` is hard coded, > and the `BUILD_USER` flags is always "frank". This commit will add > correct values for all such flags, with the exception of one, > `BUILD_FLAGS`. > > In addition, this commit will take this opportunity to clean up the > organization of the build flags. Currently, we call `add_definitions` > individually for each flag we wish to define. This commit will introduce > a new variable, `MESOS_CPPFLAGS`, which holds the build flags for the > project. We call `add_definitions` once, when `MESOS_CPPFLAGS` is > completely populated. > > > Diffs > ----- > > cmake/CompilationConfigure.cmake 1349afb44df0d922bf7f87ed1c73076590957089 > src/CMakeLists.txt c8d4260c03d8cdee1951a50d293e9fdabcd2cf84 > > Diff: https://reviews.apache.org/r/55040/diff/ > > > Testing > ------- > > > Thanks, > > Alex Clemmer > >
