Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8407 to look at the new patch set (#4). Change subject: [build] fix OPENSSL_ROOT_DIR override on RH/CentOS ...................................................................... [build] fix OPENSSL_ROOT_DIR override on RH/CentOS Due to a mistake in the condition for one if() expression, the OpenSSL location was not overridden on RedHat/CentOS machines where the OpenSSL workaround build existed in $KUDU_ROOT/thirdparty/installed/openssl-el6-workaround, even if -DOPENSSL_ROOT_DIR=... was specified for cmake invocation. Removing the level of indirection for the OPENSSL_ROOT_DIR variable in the expression fixed the issue, which seems to be in sync with https://cmake.org/cmake/help/v3.9/command/if.html At least, with this patch the issue is gone if running cmake 3.9.0. Change-Id: Ibe69b5d62ef4054e60138a154a85569d92166aff --- M CMakeLists.txt 1 file changed, 1 insertion(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/07/8407/4 -- To view, visit http://gerrit.cloudera.org:8080/8407 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ibe69b5d62ef4054e60138a154a85569d92166aff Gerrit-Change-Number: 8407 Gerrit-PatchSet: 4 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>