Hello Kudu Jenkins, Andrew Wong, Grant Henke,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/17053

to look at the new patch set (#2).

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
---
M thirdparty/build-definitions.sh
1 file changed, 6 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/17053/2
--
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: newpatchset
Gerrit-Change-Id: I10a22e1e864b19491177c25a352e2378a6f6dea3
Gerrit-Change-Number: 17053
Gerrit-PatchSet: 2
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)

Reply via email to