----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46676/#review145265 -----------------------------------------------------------
src/Makefile.am (line 535) <https://reviews.apache.org/r/46676/#comment211413> This will break agent headers. FOr instance, isolator.hpp has the following: ``` #include <mesos/slave/isolator.pb.h> ``` - Jie Yu On April 27, 2016, 1:57 a.m., zhou xing wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46676/ > ----------------------------------------------------------- > > (Updated April 27, 2016, 1:57 a.m.) > > > Review request for mesos, Kevin Klues and Vinod Kone. > > > Bugs: mesos-5230 > https://issues.apache.org/jira/browse/mesos-5230 > > > Repository: mesos > > > Description > ------- > > This patch did the following changes: > > 1. Change the 'make install' headers installation location from > '$(DESTDIR)/include/mesos/slave' to '$(DESTDIR)include/mesos/agent' > > 2. As we do not change the folder name from 'include/mesos/slave' > to 'include/mesos/agent', create a 'slave -> agent' symlink in dir > '$(DESTDIR)/include/mesos' > > > Diffs > ----- > > CHANGELOG 57f5457640c9b7ca2cb7707f2962dd0421f090f7 > src/Makefile.am e024c6d65608a55765e527a8668c415723dcfcca > > Diff: https://reviews.apache.org/r/46676/diff/ > > > Testing > ------- > > cd mesos > ./bootstrap > mkdir build > cd build > ../configure --prefix=${HOME}/install/mesos > make > make check > make install > > > Thanks, > > zhou xing > >
