----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69708/ -----------------------------------------------------------
(Updated Jan. 14, 2019, 8:21 p.m.) Review request for mesos, Benjamin Bannier, BenjaminVW BenjaminVW, Gastón Kleiman, Joseph Wu, Till Toenshoff, and Vinod Kone. Changes ------- Addressed Joseph's comment. Bugs: MESOS-9519 https://issues.apache.org/jira/browse/MESOS-9519 Repository: mesos Description ------- This patch cherry-picks an unofficial commit to disable ALPN compilation if only OpenSSL 1.0.1 is on the system. Diffs (updated) ----- 3rdparty/CMakeLists.txt 703808d063e4bba58f647b5d48b78724003bcc4e 3rdparty/grpc-1.10.0.patch 655f00387a6b308b653b23053419ec05c8b22144 3rdparty/grpc.md e06843c8b6038eb9fb809241686fd611d1daedc8 Diff: https://reviews.apache.org/r/69708/diff/4/ Changes: https://reviews.apache.org/r/69708/diff/3-4/ Testing ------- OS=ubuntu:14.04 BUILDTOOL=cmake COMPILER=gcc CONFIGURATION='--verbose --enable-libevent --enable-ssl' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1' support/docker-build.sh OS=ubuntu:16.04 BUILDTOOL=cmake COMPILER=gcc CONFIGURATION='--verbose --enable-libevent --enable-ssl' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1' support/docker-build.sh (to make sure that it works with OpenSSL 1.0.2. Also manually inspected that ALPN symbols are linked). Manually tested on Debian 9 to ensure that the CMake rule can find its OpenSSL 1.1.0. NOTE: The patch itself does not depend on r/69725, but the ubuntu:16.04 test above does. Thanks, Chun-Hung Hsiao