-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64211/
-----------------------------------------------------------

(Updated March 24, 2018, 11:39 a.m.)


Review request for mesos, Benjamin Bannier and Kevin Klues.


Changes
-------

Rebased.


Bugs: MESOS-8240
    https://issues.apache.org/jira/browse/MESOS-8240


Repository: mesos


Description
-------

An update of the discarded review /r/52543.

Works with Autotools and CMake.


Diffs (updated)
-----

  CMakeLists.txt 6702f02245e3867c06bbd9efbbf4e3b961a7d9aa 
  cmake/CompilationConfigure.cmake 64cc56ee4208afe05df0f28af5890157e4c7d82c 
  configure.ac f0f901f2e565352c2804cae7b2ac255da81ce45d 
  docs/configuration/cmake.md 1e34657ea55fa324f65f865f7d0a67084c6719d9 
  src/CMakeLists.txt 3b04e6353979c05a38f1b032704dce566d8fd561 
  src/Makefile.am a570c9f47f6de73c689930ae020fefb29b30b32d 
  src/python/cli_new/CMakeLists.txt PRE-CREATION 
  src/python/cli_new/tests/CMakeLists.txt PRE-CREATION 


Diff: https://reviews.apache.org/r/64211/diff/15/

Changes: https://reviews.apache.org/r/64211/diff/14-15/


Testing
-------

Testing done on Fedora 25.

For Autotools:
```
$ ./bootstrap
$ mkdir build
$ cd build
$ ../configure --enable-new-cli --disable-java --disable-python
$ make check
```

For CMake:
```
$ ./bootstrap
$ mkdir build
$ cd build
$ cmake .. -DENABLE_NEW_CLI=1
$ cmake --build . -- -j16
$ ./src/mesos
Mesos CLI

Usage:
  mesos (-h | --help)
  mesos --version
  mesos <command> [<args>...]

Options:
  -h --help  Show this screen.
  --version  Show version info.

Commands:
  agent   Interacts with the Mesos agents
  config  Interacts with the Mesos CLI configuration file
  task    Interacts with the tasks running in a Mesos cluster

See 'mesos help <command>' for more information on a specific command.
$ cmake --build . --target tests -- -j16
$ ctest -R CLI
Test project /home/agrillet/apache-mesos/build
    Start 4: CLITests
1/1 Test #4: CLITests .........................   Passed    3.63 sec

100% tests passed, 0 tests failed out of 1
```

Checked that the the CLI tests were run, that the content of the directory 
`build/src/cli` was as expected, and that `build/src/mesos` was correctly 
running.


Thanks,

Armand Grillet

Reply via email to