Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9931 )
Change subject: WIP: use ExternalProject_Add to define thirdparty deps ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/9931/1/CMakeLists.txt File CMakeLists.txt: http://gerrit.cloudera.org:8080/#/c/9931/1/CMakeLists.txt@836 PS1, Line 836: if(${LIB_NAME} STREQUAL "snappy") > Makes sense. That might also allow us to have separate ExternalProject tar I don't think we want to handle TSAN by defining different targets; it should suffice to vary the external project's build/install directories. But I'm not 100% sure as I haven't made it that far yet. -- To view, visit http://gerrit.cloudera.org:8080/9931 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4dec48958bf09f8f1a0d8af08c815a22d8240c3d Gerrit-Change-Number: 9931 Gerrit-PatchSet: 1 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Thu, 05 Apr 2018 18:01:17 +0000 Gerrit-HasComments: Yes
