Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10373 )
Change subject: KUDU-2346: Fix mac pkg-config issue. ...................................................................... KUDU-2346: Fix mac pkg-config issue. Previously an empty return string from pkg-config --cflags was used as a proxy for testing if the package was installed. The MacOs version of pkg-config returns an error message in response to this command when a package is not installed. The fix is to property test the return value of pkg-config --cflags. Change-Id: I8a6b543dc65ac65d0dedf7e39f735b4805612bf1 Reviewed-on: http://gerrit.cloudera.org:8080/9652 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Alexey Serbin <[email protected]> (cherry picked from commit 9f6b7f16828250a89e0e331026eeb05687c6eeab) Reviewed-on: http://gerrit.cloudera.org:8080/10373 Tested-by: Kudu Jenkins --- M thirdparty/build-thirdparty.sh 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/10373 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.7.x Gerrit-MessageType: merged Gerrit-Change-Id: I8a6b543dc65ac65d0dedf7e39f735b4805612bf1 Gerrit-Change-Number: 10373 Gerrit-PatchSet: 3 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: [email protected]
