mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Clemmer" <clemmer.alexan...@gmail.com>
Subject Re: Review Request 36740: Add small fixes allowing CMake to build Stout tests on OS X 10.10
Date Thu, 23 Jul 2015 23:41:00 GMT

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

(Updated July 23, 2015, 11:41 p.m.)


Review request for mesos, Benjamin Hindman, Artem Harutyunyan, and Joris Van Remoortere.


Repository: mesos


Description
-------

There are two things that have to change in the current CMake build
solution in order to work on OS X 10.10:
  * librt (which holds the POSIX.1b Realtime Extensions interfaces in
    Linux, including things like async I/O and memory-mapped files, etc.)
    does not exist on OS X, and does not need to be linked
  * The version of GTest attempts to find std::tr1::tuple, which is not
    available on OS X 10.10. Like we have done in autotools (see[1] and
    also configure.ac) we simply define `GTEST_USE_OWN_TR1_TUPLE` when
    we're compiling on OS X.

[1] https://mail-archives.apache.org/mod_mbox/mesos-dev/201403.mbox/%3C20140306223933.2625.25724@reviews.apache.org%3E


Diffs
-----

  3rdparty/libprocess/3rdparty/stout/cmake/FindSvn.cmake c0c0f347952286f67f9e90a922f8f0ead41ed602

  3rdparty/libprocess/3rdparty/stout/cmake/StoutTestsConfigure.cmake e9be4aa71f44f9502a20f4b1fe09bd33bbe4cf71


Diff: https://reviews.apache.org/r/36740/diff/


Testing
-------

Tested fixes on OS X 10.10 and Ubuntu 14 boxes.


Thanks,

Alex Clemmer


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