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


Patch looks great!

Reviews applied: [36819]

All tests passed.

- Mesos ReviewBot


On Aug. 13, 2015, 6:46 p.m., haosdent huang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/36819/
> -----------------------------------------------------------
> 
> (Updated Aug. 13, 2015, 6:46 p.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 b9ecafd1813d87d604364d354994e4dba386f958 
>   src/Makefile.am 111aed92820689b12ee4073269ce34db7be30960 
>   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/setup.py.in 880e2a6fe4470e50bbfa805967fb983fb8a9987f 
>   src/python/interface/src/mesos/__init__.py 
> f48ad10528712b2b8960f1863d156b88ed1ce311 
>   src/python/native/setup.py.in 9fc9ad2bdeaf1ad3f007e36f24bbe5b10dedc77b 
>   src/python/native/src/mesos/__init__.py 
> f48ad10528712b2b8960f1863d156b88ed1ce311 
>   src/python/protocol/setup.py.in 72cb770bda659957c03853c3c99ba6c53f606df8 
>   src/python/protocol/src/mesos/__init__.py 
> f48ad10528712b2b8960f1863d156b88ed1ce311 
>   src/python/setup.py.in 8ffde68e60e8c968f196e6e0790aeef47d0fb662 
> 
> 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
> 
>

Reply via email to