Mike Percy has posted comments on this change.

Change subject: cmake: Throw cmake error if unit test does not exist
......................................................................


Patch Set 2:

This is what the error output looked like when I defined a non-existent test 
called consistency-itestxxxx:

CMake Error at CMakeLists.txt:659 (message):
  Neither consistency-itestxxxx nor consistency-itestxxxx.cc were found in
  /home/mpercy/src/kudu/src/kudu/integration-tests/
Call Stack (most recent call first):
  src/kudu/integration-tests/CMakeLists.txt:53 (ADD_KUDU_TEST)


-- Configuring incomplete, errors occurred!
See also "/home/mpercy/src/kudu/build/dynclang/CMakeFiles/CMakeOutput.log".
FAILED: /home/mpercy/src/kudu/thirdparty/installed/common/bin/cmake 
-H/home/mpercy/src/kudu -B/home/mpercy/src/kudu/build/dynclang

-- 
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: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-HasComments: No

Reply via email to