Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9652 )
Change subject: KUDU-2346: Fix mac pkg-config/PKG_CONFIG_PATH issue. ...................................................................... Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/9652/4/thirdparty/build-thirdparty.sh File thirdparty/build-thirdparty.sh: http://gerrit.cloudera.org:8080/#/c/9652/4/thirdparty/build-thirdparty.sh@174 PS4, Line 174: if (( 1 -eq $(pkg-config --exists openssl) )); then What if pkg-config returns anything else but 1 and 0 (e.g. no pkg-config around)? Why not just: if ! OPENSSL_CFLAGS=$(pkg-config --cflags openssl); then # If OpenSSL is built via Homebrew, pkg-config does not report on cflags. homebrew_openssl_dir=/usr/local/opt/openssl if [ -d $homebrew_openssl_dir ]; then OPENSSL_CFLAGS="-I$homebrew_openssl_dir/include" OPENSSL_LDFLAGS="-L$homebrew_openssl_dir/lib" fi fi -- To view, visit http://gerrit.cloudera.org:8080/9652 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8a6b543dc65ac65d0dedf7e39f735b4805612bf1 Gerrit-Change-Number: 9652 Gerrit-PatchSet: 4 Gerrit-Owner: [email protected] Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: [email protected] Gerrit-Comment-Date: Fri, 16 Mar 2018 17:31:20 +0000 Gerrit-HasComments: Yes
