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

(Updated June 16, 2018, 2:06 a.m.)


Review request for mesos, Andrew Schwartzmeyer, Benjamin Bannier, Joseph Wu, 
and Jan Schlicht.


Changes
-------

Added macOS detection, added LIBEVENT_ROOT_DIR flag, updated cmake 
documentation.


Summary (updated)
-----------------

Allowed for unbundled libevent cmake builds.


Bugs: MESOS-8998
    https://issues.apache.org/jira/browse/MESOS-8998


Repository: mesos


Description (updated)
-------

Adds the option UNBUNDLED_LIBEVENT for using an installed libevent
instead of the bundled version. That option is set towards
TRUE by default on macOS, FALSE otherwise. Additionally adds the flag
LIBEVENT_ROOT_DIR for specifying the path to an unbundled libevent.
Updates the cmake documentation with the above flags.


Diffs (updated)
-----

  3rdparty/CMakeLists.txt 3c1dfcc7885693cd76331b515ca8e361bd1976d0 
  3rdparty/cmake/Findlibevent.cmake PRE-CREATION 
  cmake/CompilationConfigure.cmake 2f92acb3a140faa48e3639c7000be7f43020ad7d 
  docs/configuration/cmake.md 74abe65507d251ffb9cbae31a6fa18eb0d76e79b 


Diff: https://reviews.apache.org/r/67606/diff/2/

Changes: https://reviews.apache.org/r/67606/diff/1-2/


Testing
-------

```
$ cmake .. -DENABLE_LIBEVENT=TRUE -DENABLE_SSL=TRUE 
-DOPENSSL_ROOT_DIR=/usr/local/opt/openssl
$ cmake --build . --target tests -- -j6

$ otool -L 3rdparty/libprocess/src/libprocess.dylib
libprocess.dylib:
        @rpath/libprocess.dylib (compatibility version 0.0.0, current version 
0.0.0)
        /usr/local/opt/openssl/lib/libssl.1.0.0.dylib (compatibility version 
1.0.0, current version 1.0.0)
        /usr/local/opt/openssl/lib/libcrypto.1.0.0.dylib (compatibility version 
1.0.0, current version 1.0.0)
        
/Users/till/Development/mesos/build/3rdparty/libevent-2.1.5-beta/src/libevent-2.1.5-beta-build/lib/libevent.2.1.5.dylib
 (compatibility version 2.1.5, current version 0.0.0)
        /usr/local/opt/apr/libexec/lib/libapr-1.0.dylib (compatibility version 
7.0.0, current version 7.3.0)
        /usr/lib/libcurl.4.dylib (compatibility version 7.0.0, current version 
9.0.0)
        
/Users/till/Development/mesos/build/3rdparty/glog-0.3.3/src/glog-0.3.3-build/lib/libglog.0.dylib
 (compatibility version 1.0.0, current version 1.0.0)
        
/Users/till/Development/mesos/build/3rdparty/protobuf-3.5.0/src/protobuf-3.5.0-build/libprotobuf.dylib
 (compatibility version 0.0.0, current version 0.0.0)
        /usr/lib/libz.1.dylib (compatibility version 1.0.0, current version 
1.2.11)
        /usr/local/opt/subversion/lib/libsvn_delta-1.0.dylib (compatibility 
version 1.0.0, current version 1.0.0)
        /usr/local/opt/subversion/lib/libsvn_diff-1.0.dylib (compatibility 
version 1.0.0, current version 1.0.0)
        /usr/local/opt/subversion/lib/libsvn_subr-1.0.dylib (compatibility 
version 1.0.0, current version 1.0.0)
        /usr/lib/libc++.1.dylib (compatibility version 1.0.0, current version 
400.9.3)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current 
version 1252.200.5)
```

```
$ cmake .. -DENABLE_LIBEVENT=TRUE -DENABLE_SSL=TRUE 
-DOPENSSL_ROOT_DIR=/usr/local/opt/openssl -DUNBUNDLED_LIBEVENT=TRUE
$ cmake --build . --target tests -- -j6

$ otool -L 3rdparty/libprocess/src/libprocess.dylib
libprocess.dylib:
        @rpath/libprocess.dylib (compatibility version 0.0.0, current version 
0.0.0)
        /usr/local/opt/openssl/lib/libssl.1.0.0.dylib (compatibility version 
1.0.0, current version 1.0.0)
        /usr/local/opt/openssl/lib/libcrypto.1.0.0.dylib (compatibility version 
1.0.0, current version 1.0.0)
        /usr/local/opt/libevent/lib/libevent-2.0.5.dylib (compatibility version 
7.0.0, current version 7.9.0)
        /usr/local/opt/apr/libexec/lib/libapr-1.0.dylib (compatibility version 
7.0.0, current version 7.3.0)
        /usr/lib/libcurl.4.dylib (compatibility version 7.0.0, current version 
9.0.0)
        
/Users/till/Development/mesos/build/3rdparty/glog-0.3.3/src/glog-0.3.3-build/lib/libglog.0.dylib
 (compatibility version 1.0.0, current version 1.0.0)
        
/Users/till/Development/mesos/build/3rdparty/protobuf-3.5.0/src/protobuf-3.5.0-build/libprotobuf.dylib
 (compatibility version 0.0.0, current version 0.0.0)
        /usr/lib/libz.1.dylib (compatibility version 1.0.0, current version 
1.2.11)
        /usr/local/opt/subversion/lib/libsvn_delta-1.0.dylib (compatibility 
version 1.0.0, current version 1.0.0)
        /usr/local/opt/subversion/lib/libsvn_diff-1.0.dylib (compatibility 
version 1.0.0, current version 1.0.0)
        /usr/local/opt/subversion/lib/libsvn_subr-1.0.dylib (compatibility 
version 1.0.0, current version 1.0.0)
        /usr/local/opt/libevent/lib/libevent_openssl-2.0.5.dylib (compatibility 
version 7.0.0, current version 7.9.0)
        /usr/local/opt/libevent/lib/libevent_pthreads-2.0.5.dylib 
(compatibility version 7.0.0, current version 7.9.0)
        /usr/lib/libc++.1.dylib (compatibility version 1.0.0, current version 
400.9.3)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current 
version 1252.200.5)
```


Thanks,

Till Toenshoff

Reply via email to