----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62731/ -----------------------------------------------------------
(Updated Oct. 2, 2017, 11:58 a.m.) Review request for mesos, Benjamin Bannier, Jeff Coffler, John Kordich, Joseph Wu, and Li Li. Repository: mesos Description ------- This was a magic variable that was used to add compiler definitions globally. Instead, global definitions are now added explicitly with `add_definitions`, and others with `target_compile_definitions`. Diffs ----- cmake/CompilationConfigure.cmake 4cb8dddf93694722af2377c4a55a195260c8a0cb src/CMakeLists.txt 1a828c4351ded36f51ccbbe67147da2f50b9cdb1 Diff: https://reviews.apache.org/r/62731/diff/1/ Testing (updated) ------- This was the last patch which in this series which touched files other than documentation. Tested the regular test targets on Windows with CMake with a clean build, everything still builds and passes. Same on Linux. Thanks, Andrew Schwartzmeyer