> On Oct. 12, 2015, 7:06 p.m., Joseph Wu wrote: > > CMakeLists.txt, lines 96-99 > > <https://reviews.apache.org/r/39093/diff/2/?file=1094407#file1094407line96> > > > > Still only runs the libprocess tests. > > > > This runs both tests: > > ``` > > add_custom_target( > > check ${STOUT_TESTS_TARGET} > > COMMAND ${PROCESS_TESTS_TARGET} > > DEPENDS ${PROCESS_TESTS_TARGET} ${STOUT_TESTS_TARGET} > > ) > > ``` > > > > So for future tests, you'll need to insert another `COMMAND > > ${..._TESTS_TARGET}`.
Oh man, sorry, the process tests fail on corpnet, and in the flurry of commits I forgot to come back and verify this worked when I went home. - Alex ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39093/#review102295 ----------------------------------------------------------- On Oct. 10, 2015, 4:27 a.m., Alex Clemmer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39093/ > ----------------------------------------------------------- > > (Updated Oct. 10, 2015, 4:27 a.m.) > > > Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph > Wu. > > > Bugs: MESOS-3597 > https://issues.apache.org/jira/browse/MESOS-3597 > > > Repository: mesos > > > Description > ------- > > CMake:[3/3] Add `make check` target. > > > Diffs > ----- > > CMakeLists.txt a5a66c13496a95bb7095504a4645909d887fb49f > > Diff: https://reviews.apache.org/r/39093/diff/ > > > Testing > ------- > > > Thanks, > > Alex Clemmer > >
