Adar Dembo has posted comments on this change. Change subject: cmake: Throw cmake error if unit test does not exist ......................................................................
Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/5222/1/CMakeLists.txt File CMakeLists.txt: Line 658: else() Would be nice to say that neither REL_TEST_NAME nor REL_TEST_NAME.cc were found. PS1, Line 659: {.cc} Are these braces redundant? -- To view, visit http://gerrit.cloudera.org:8080/5222 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6c7d048f44624c8406198a098e0a7ecb44f8a08e Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes
