mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Avinash sridharan <avin...@mesosphere.io>
Subject Review Request 48612: Removed conditional for of setting `--prefix=/`.
Date Mon, 13 Jun 2016 05:15:56 GMT

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

Review request for mesos and Kapil Arya.


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


Repository: mesos


Description
-------

For bundled 3rdparty packages the `--prefix` flag should always be set
to `/`, else the 3rdparty installation path might append the
$MESOS_INSTALL to the installation path.


Diffs
-----

  3rdparty/Makefile.am fb4a37d50e751703b4ccddb0e004b58560707067 

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


Testing
-------

make and make install

Bundled zookeeper installation takes place on the following path:
vagrant@mesos-dev:~/mesosphere/mesos/build/3rdparty$ find ~/mesos_install/ -name "zookeeper*"
/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