Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10311 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/10311 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- M thirdparty/build-definitions.sh M thirdparty/preflight.py 2 files changed, 9 insertions(+), 13 deletions(-) Approvals: Kudu Jenkins: Verified Todd Lipcon: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I5fdae9c77e7d8b6d11b6dc2997686bbdbb383301 Gerrit-Change-Number: 10311 Gerrit-PatchSet: 2 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
