> On Aug. 15, 2014, 6:05 p.m., Vinod Kone wrote: > > 3rdparty/libprocess/configure.ac, line 111 > > <https://reviews.apache.org/r/24734/diff/3/?file=661496#file661496line111> > > > > should this be --with-http_parser? "_" between http and parser. > > Timothy St. Clair wrote: > why? > > $ sudo yum list | grep http | grep parse > http-parser.x86_64 2.0-5.20121128gitcd01361.fc20 > @fedora > http-parser-devel.x86_64 2.0-5.20121128gitcd01361.fc20 > @fedora > > Vinod Kone wrote: > oh, because the test below "if test -n "`echo $with_http_parser`" changed > the dash to underscore. i guess that works because of autotools magic?
autotool will replace - with _ > On Aug. 15, 2014, 6:05 p.m., Vinod Kone wrote: > > src/zookeeper/zookeeper.hpp, lines 36-38 > > <https://reviews.apache.org/r/24734/diff/3/?file=661502#file661502line36> > > > > reorder. I've revert this change since it does not work with our bundled zk. > On Aug. 15, 2014, 6:05 p.m., Vinod Kone wrote: > > configure.ac, line 412 > > <https://reviews.apache.org/r/24734/diff/3/?file=661497#file661497line412> > > > > shouldn't this be "$PROTOBUFPREFIX/bin" because that's what > > AC_CHECK_TOOL is checking below? AC_CHECK_TOOL will search PATH. the last argument is the PATH. I.e., we are searching protoc command under prefix/bin > 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? > > Timothy St. Clair wrote: > It's strictly to prefent name collisions.. > > PROTOBUF_COMPILER could probably be PROTOBUF_COMPILER_TEST. Consolidated. > On Aug. 15, 2014, 6:05 p.m., Vinod Kone wrote: > > 3rdparty/libprocess/configure.ac, line 423 > > <https://reviews.apache.org/r/24734/diff/3/?file=661496#file661496line423> > > > > maybe the error here could also say "please set PROTOBUF_JAR > > environment variable" ? Hum, user might set the environment variable, but a wrong path. I prefer keep the existing:) - Jie ----------------------------------------------------------- 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 > >
