> On Aug. 15, 2014, 6:05 p.m., Vinod Kone wrote: > > 3rdparty/libprocess/configure.ac, line 425 > > <https://reviews.apache.org/r/24734/diff/3/?file=661496#file661496line425> > > > > I'm little confused on what the difference is between PROTOCOMPILER vs > > PROTOBUF_COMPILER vs PROTOC. can we consolidate?
It's strictly to prefent name collisions.. PROTOBUF_COMPILER could probably be PROTOBUF_COMPILER_TEST. - Timothy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24734/#review50751 ----------------------------------------------------------- On Aug. 15, 2014, 5:19 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24734/ > ----------------------------------------------------------- > > (Updated Aug. 15, 2014, 5:19 p.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 7ba4fad65a144bfe832e75993e5b8f72fcd66cc6 > configure.ac 30a7e267dac025270bb69b9ae498cbb8b3f0392c > src/tests/group_tests.cpp b265bcba3bfc46e80323ba49cb9193457b36d28c > src/tests/master_contender_detector_tests.cpp > fdddfa1a9c793bb406d85d214a18ca4ced53b09f > src/tests/zookeeper_tests.cpp 85886199e4d8872af9d944e0f26377b9d85a8c12 > src/zookeeper/authentication.hpp 7b6b767b1451bb361c0c26675b642a072f6e5103 > src/zookeeper/zookeeper.hpp 6d21ed8aba85de1ad1302581fddbc09d22ebe54b > > 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 > >
