mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jan Schlicht <>
Subject Re: Review Request 66330: Added 'OPENSSL_INCLUDE_DIR' to forwarded arguments.
Date Wed, 28 Mar 2018 12:01:15 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated March 28, 2018, 2:01 p.m.)

Review request for mesos, Andrew Schwartzmeyer and Benjamin Bannier.


Added a comment.

Repository: mesos


To help third-party dependencies find and build with OpenSSL, certain
options are forwarded to them. Dependencies using the 'FindOpenSSL'
module need either 'OPENSSL_ROOT_DIR' or 'OPENSSL_INCLUDE_DIR' to figure
out the location of OpenSSL. In the module 'OPENSSL_ROOT_DIR' acts as a
hint for 'OPENSSL_INCLUDE_DIR'. I.e., 'OPENSSL_INCLUDE_DIR' will be set
by the module. As a consequence, if 'OPENSSL_ROOT_DIR' is not configured
in 'ccmake', users will only be prompted to provide
'OPENSSL_INCLUDE_DIR'. By forwarding 'OPENSSL_INCLUDE_DIR' we cover this
case for third-party dependencies, thus allowing users to configure the
project in 'ccmake' without having to explicitly set 'OPENSSL_ROOT_DIR'.

Diffs (updated)

  3rdparty/CMakeLists.txt 2b63b58f7d6a88c9986b746283dcfa79b7bcb270 




On macOS 10.13.3 with OpenSSL installed via Homebrew:

$ mkdir build && cd build
$ ccmake -GNinja ..

* Press "c" to configure.
* Press "c" to configure. There will be an error explaining that 'OPENSSL_ROOT_DIR' or 'OPENSSL_INCLUDE_DIR'
is missing.
* `ccmake` will ask for "OPENSSL_INCLUDE_DIR", set it to `/usr/local/opt/openssl/include`
(assuming `$(brew --prefix) == /usr/local`)
* Press "c" to configure again. There shouldn't be an error now.
* Press "g" to generate the configuration.

$ ninja


Jan Schlicht

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