> On Aug. 15, 2014, 2:23 p.m., Timothy St. Clair wrote: > > configure.ac, line 283 > > <https://reviews.apache.org/r/24734/diff/1/?file=661315#file661315line283> > > > > This needs to be here or else the includes fail on distributions. > > Jie Yu wrote: > Are you saying that I should do: > > CPPFLAGS="$CPPFLAGS -I/usr/include" here? > > Is that the default? I'm curious what the error is?
The build fails on zookeeper.h includes inside of the mesos srouce. b/c the -I path is only /usr/include and zookeeper lives under '/usr/include/zookeeper' - Timothy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24734/#review50719 ----------------------------------------------------------- On Aug. 15, 2014, 6:36 a.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24734/ > ----------------------------------------------------------- > > (Updated Aug. 15, 2014, 6:36 a.m.) > > > Review request for mesos, Timothy St. Clair and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > Merged the fix in https://reviews.apache.org/r/24714/ > > Fixed a few other issues. > > Also, did a few consistency/style fix. > > > Diffs > ----- > > 3rdparty/libprocess/configure.ac f552e60cbac460a488bf1cd36ddd1e1a54190634 > configure.ac 30a7e267dac025270bb69b9ae498cbb8b3f0392c > > Diff: https://reviews.apache.org/r/24734/diff/ > > > Testing > ------- > > On my mac: > > CC=gcc-4.8 CXX=g++-4.8 ../../../mesos/configure > --prefix=/Users/jyu/workspace/mesos-dist/default --disable-bundled > --with-leveldb=/opt/twitter --with-glog=/opt/twitter > --with-zookeeper=/tmp/zookeeper/3.4.6 --with-protobuf=/tmp/protobuf/2.5.0 > --disable-python --with-gmock=/tmp/gmock/gmock-1.7.0 > --with-libev=/opt/twitter --with-picojson=/tmp/picojson > --with-http-parser=/tmp/http-parser > > > Thanks, > > Jie Yu > >
