mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Avinash sridharan <avin...@mesosphere.io>
Subject Re: Review Request 48594: Added bundled zookeeper to the module-dependencies.
Date Mon, 13 Jun 2016 04:56:39 GMT

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

(Updated June 13, 2016, 4:56 a.m.)


Review request for mesos, Jie Yu and Kapil Arya.


Changes
-------

Updated the test description with results after running `make clean` in build/3rdparty.


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


Repository: mesos


Description
-------

Modules that want to use the `State` API for storing information in
the replicated log require zookeeper client headers. With this change
the bundled zookeeper client headers will be installed along with the
Mesos libraries and headers under the Mesos installation path.


Diffs
-----

  3rdparty/Makefile.am fb4a37d50e751703b4ccddb0e004b58560707067 

Diff: https://reviews.apache.org/r/48594/diff/


Testing (updated)
-------

cd build/3rdparty; make clean
cd $MESOS_SRC/build/
make and make install

After running make install made sure that the bundled client libraries are installed under
the $MESOS_INSTALL path. Here is an example output:

/home/vagrant/mesos_install/lib/mesos/3rdparty/include/zookeeper
/home/vagrant/mesos_install/lib/mesos/3rdparty/include/zookeeper/zookeeper.h
/home/vagrant/mesos_install/lib/mesos/3rdparty/include/zookeeper/zookeeper_version.h
/home/vagrant/mesos_install/lib/mesos/3rdparty/include/zookeeper/zookeeper_log.h
/home/vagrant/mesos_install/lib/mesos/3rdparty/include/zookeeper/zookeeper.jute.h
/home/vagrant/mesos_install/include/mesos/zookeeper
/home/vagrant/mesos_install/include/mesos/zookeeper/zookeeper.hpp
/home/vagrant/mesos_install/include/mesos/state/zookeeper.hpp


Thanks,

Avinash sridharan


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message