Hello Todd Lipcon,

I'd like you to do a code review. Please visit

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

to review the following change.


Change subject: thirdparty: build llvm against thirdparty zlib
......................................................................

thirdparty: build llvm against thirdparty zlib

Turns out we already have zlib in thirdparty, so let's use it when building
LLVM. For some reason LLVM doesn't actually enable zlib support (i.e. the
llvm::zlib functions in libLLVMSupport.a are still stubs), but this lets
IWYU build even if the host doesn't have zlib-devel installed.

Change-Id: I5fdae9c77e7d8b6d11b6dc2997686bbdbb383301
---
M thirdparty/build-definitions.sh
M thirdparty/preflight.py
2 files changed, 9 insertions(+), 13 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/11/10311/1
--
To view, visit http://gerrit.cloudera.org:8080/10311
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5fdae9c77e7d8b6d11b6dc2997686bbdbb383301
Gerrit-Change-Number: 10311
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to