----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66314/#review200305 -----------------------------------------------------------
3rdparty/CMakeLists.txt Lines 359-360 (original), 365-366 (patched) <https://reviews.apache.org/r/66314/#comment281001> If I understood the above discussion correctly, I think the following would work well: Use `${CMAKE_MAKE_PROGRAM}` and `${CMAKE_MAKE_PROGRAM} install` for the BUILD/INSTALL commands, instead of adding `${GNU_MAKE}`. Then in the MesosConfigure.cmake, we can have a platform check `if (FREEBSD) ... make sure CMAKE_MAKE_PROGRAM is gmake` or something to that effect. - Andrew Schwartzmeyer On April 2, 2018, 11:36 a.m., David Forsythe wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66314/ > ----------------------------------------------------------- > > (Updated April 2, 2018, 11:36 a.m.) > > > Review request for mesos, Andrew Schwartzmeyer and Benjamin Bannier. > > > Bugs: MESOS-4176 > https://issues.apache.org/jira/browse/MESOS-4176 > > > Repository: mesos > > > Description > ------- > > Fix 3rdparty build commands for FreeBSD. > > > Diffs > ----- > > 3rdparty/CMakeLists.txt 2b63b58f7d6a88c9986b746283dcfa79b7bcb270 > cmake/CompilationConfigure.cmake 64cc56ee4208afe05df0f28af5890157e4c7d82c > > > Diff: https://reviews.apache.org/r/66314/diff/1/ > > > Testing > ------- > > cmake --build on FreeBSD > > > Thanks, > > David Forsythe > >