----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36819/#review95303 -----------------------------------------------------------
Ship it! LGTM module the issue below. BTW, do we know if anyone is using the `mesos-cli` tool? I had a discussion with ThomasR a while ago and he suggested that the `mesos-cli` tool should be removed from mesos source repo entirely. src/cli/python/mesos/__init__.py (line 25) <https://reviews.apache.org/r/36819/#comment150189> Don't we need a `mesos.cli` here as well? - Kapil Arya On Aug. 7, 2015, 11:08 a.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36819/ > ----------------------------------------------------------- > > (Updated Aug. 7, 2015, 11:08 a.m.) > > > Review request for mesos, Benjamin Hindman, Bernd Mathiske, and Marco > Massenzio. > > > Bugs: MESOS-3149 > https://issues.apache.org/jira/browse/MESOS-3149 > > > Repository: mesos > > > Description > ------- > > Use setup.py in python cli package. > > > Diffs > ----- > > Makefile.am f8e958d376efa55ace7c5727a31b8e747403641e > bin/mesos.sh.in 5cbeac4330a9f45fc6d54b8c2d383f48e4098f95 > configure.ac 0794279dd2e23b5b593e7e388bd6d04e17c746a6 > src/Makefile.am 35ebbbd0bd9c9dd059c02ce3dc22c780b929be81 > src/cli/python/mesos/__init__.py 028b0d27fb193bac96f2a6a3201ee4cc8fd369ef > src/cli/python/mesos/cli.py > src/cli/python/mesos/futures.py > src/cli/python/mesos/http.py > src/python/cli/src/mesos/__init__.py PRE-CREATION > src/python/interface/src/mesos/__init__.py > f48ad10528712b2b8960f1863d156b88ed1ce311 > src/python/native/src/mesos/__init__.py > f48ad10528712b2b8960f1863d156b88ed1ce311 > src/python/protocol/src/mesos/__init__.py > f48ad10528712b2b8960f1863d156b88ed1ce311 > > Diff: https://reviews.apache.org/r/36819/diff/ > > > Testing > ------- > > make check > > # Test in CentOS 6.6, OS X 10.10, Ubuntu 14.04 > ## test steps: > 1. sudo make install > 2. export PYTHONPATH > 3. python -c 'import mesos; from mesos import cli, http, futures' > > > Thanks, > > haosdent huang > >