mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Forsythe <dforsy...@gmail.com>
Subject Re: Review Request 66314: Fix 3rdparty build commands for FreeBSD.
Date Mon, 02 Apr 2018 18:54:21 GMT


> On April 2, 2018, 6:42 p.m., Andrew Schwartzmeyer wrote:
> > 3rdparty/CMakeLists.txt
> > Lines 359-360 (original), 365-366 (patched)
> > <https://reviews.apache.org/r/66314/diff/1/?file=1989124#file1989124line365>
> >
> >     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.

If we do that, we would be forcing gmake for the entire build, right? It's not clear to me
that is the best approach, since the only problem is leveldb (and since newer versions of
leveldb seem to use cmake?). I might even be more incline to try patching LevelDBs Makefile
rather than force that.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66314/#review200305
-----------------------------------------------------------


On April 2, 2018, 6:36 p.m., David Forsythe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/66314/
> -----------------------------------------------------------
> 
> (Updated April 2, 2018, 6:36 p.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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message