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") > This would all go away when we do this in the large, right? This was just a hack I used for the POC, but we will need logic to establish a dependency between "foolib" (the library target in Kudu) with "foolib-dep" (the target created by ExternalProject_Add). -- 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 03:39:17 +0000 Gerrit-HasComments: Yes
