> On July 27, 2015, 5:58 p.m., Marco Massenzio wrote: > > thanks for doing this! > > (this may also fix a long-standing issue of mine: > > https://issues.apache.org/jira/browse/MESOS-2337 - could you please take a > > look?) > > > > Only a few minor nits about formatting and stuff, then I think this is good. > > We'll need to find a committer to push this. > > > > Did you test it? on which platforms?
Hi, @marco. Thank you for your review. I test it on CentOS 6.6. Could you review this again? Thank you very much. - haosdent ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36819/#review93141 ----------------------------------------------------------- On July 29, 2015, 7:49 a.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36819/ > ----------------------------------------------------------- > > (Updated July 29, 2015, 7:49 a.m.) > > > Review request for mesos, Benjamin Hindman 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 546c9bbf775a4ef481fafb3a58c85c6d80e19500 > src/Makefile.am 0794969b4b147e0f837006f2f2eba6c6b28eb332 > 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 > > > Thanks, > > haosdent huang > >