----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66314/#review200098 -----------------------------------------------------------
3rdparty/CMakeLists.txt Lines 58-63 (patched) <https://reviews.apache.org/r/66314/#comment280742> Like discussed offline, I don't think there is a reason we need to bolt such logic on the used system, at least currently. Let's instead use `find_program` to find a `make` and use it to set this variable. Unless I miss something, we do not depend on GNU make here, so let's maybe reflect that in a more general name, e.g., just `MAKE` if it is available. - Benjamin Bannier On March 27, 2018, 9:10 p.m., David Forsythe wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66314/ > ----------------------------------------------------------- > > (Updated March 27, 2018, 9:10 p.m.) > > > Review request for mesos 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 > >
