Hello Dan Burkert, Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/4549
to review the following change.
Change subject: thirdparty: clean up llvm cmake files too
......................................................................
thirdparty: clean up llvm cmake files too
The upgrade to LLVM 3.9 changed the list of targets provided in LLVM's cmake
files. If the cmake files are left behind and make it into an LLVM 3.8-based
build, the subsequent Kudu build will fail because it can't find all of the
targets listed.
It's too late to help with the LLVM 3.9 upgrade, but hopefully it'll ease
future upgrades.
Change-Id: I98393cf1f8afc2ced78245cad5e2e24ee9410214
---
M thirdparty/build-definitions.sh
1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/4549/1
--
To view, visit http://gerrit.cloudera.org:8080/4549
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I98393cf1f8afc2ced78245cad5e2e24ee9410214
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
