Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17053 )
Change subject: [thirdparty] use system curl for cmake ...................................................................... [thirdparty] use system curl for cmake Since building Kudu from source requires curl to be installed, let's rely on that system package when building cmake. In addition to reducing the build time of the third-party components, it also helps avoiding build failures in environments where multiple OpenSSL versions are installed. I guess a proper fix would be updating cmake configure/build scripts and makefiles, but I'm not sure it's worth it: removing an extra component from the build gives the benefit of shorter build times. Change-Id: I10a22e1e864b19491177c25a352e2378a6f6dea3 Reviewed-on: http://gerrit.cloudera.org:8080/17053 Tested-by: Kudu Jenkins Reviewed-by: Grant Henke <[email protected]> --- M thirdparty/build-definitions.sh 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Kudu Jenkins: Verified Grant Henke: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17053 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I10a22e1e864b19491177c25a352e2378a6f6dea3 Gerrit-Change-Number: 17053 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
